linux: Use the input_event_X macros to access input_event members
Description
Use the input_event_sec and input_event_usec members (macros in C) on all platforms, no matter whether time_t is 32 or 64 bits.
This means the same members are available in the struct no matter whether it is a 32 or 64 bit platform or the linux_time_bits64 config is set.
This is a breaking change.
Sources
https://github.com/torvalds/linux/blob/0cc53520e68bea7fb80fdc6bdf8d226d1b6a98d9/include/uapi/linux/input.h#L28
Checklist
- [x] Relevant tests in
libc-test/semverhave been updated - [x] No placeholder or unstable values like
*LASTor*MAXare included (see #3131) - [x] Tested locally (
cd libc-test && cargo test --target mytarget); especially relevant for platforms that may not be checked in CI
This is a breaking change and should not be backported to the 0.2 branch.
Reminder, once the PR becomes ready for a review, use @rustbot ready.
I've seen your comments and hope to get around to this next week.
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.
Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.
@rustbot ready
@tgross35 The change causes CI errors for i686-linux-unknown-musl. I was not sure what the right thing to do was - I have not been keeping up to date on changes - so I added another time64 exception for musl.
The musl time64 changes caused a conflict here so this will need a rebase (sorry about that)
@tgross35 The change causes CI errors for
i686-linux-unknown-musl. I was not sure what the right thing to do was - I have not been keeping up to date on changes - so I added another time64 exception for musl.
I think it should be fixed by https://github.com/rust-lang/libc/pull/4463 which resolved everything else in that block (hence the conflict), musl on relevant platforms should be setting the linux time64 cfg now.
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.
Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.
Rebase is done, I think the failing check is unrelated.
@rustbot ready
Thank you!
Since this is breaking, @rustbot label +stable-declined