Fix size of time_t in vxworks#4268
Conversation
|
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @tgross35 (or someone else) some time within the next two weeks. Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (
|
|
@rustbot label stable-nominated |
|
Cc @jackorobot @biabbas would you be able to double check this? I know there were some surprising differences on vxworks, from #4043. |
|
@tgross35 |
tgross35
left a comment
There was a problem hiding this comment.
Thanks for confirming. Looks like this has existed since #1408 (cc @BaoshanPang)
(backport <rust-lang#4268>) (cherry picked from commit eec39f5)
(backport <rust-lang#4268>) (cherry picked from commit eec39f5)
Description
This fixes the size of time_t in VxWorks which was incorrectly set to c_long. This broke compatibility with 32-bit targets.
Requesting stable-nomination.
Sources
VxWorks sources are copyrighted.
Checklist
libc-test/semverhave been updated*LASTor*MAXareincluded (see #3131)