alexcrichton / filetime

Accessing file timestamps in a platform-agnostic fashion in Rust
Apache License 2.0
122 stars 56 forks source link

Update use of libc::timespec to prepare for future libc version #91

Closed wesleywiser closed 1 year ago

wesleywiser commented 1 year ago

In a future release of the libc crate, libc::timespec will contain private padding fields on *-linux-musl targets and so the struct will no longer be able to be created using the literal initialization syntax.

Update struct literal use of libc::timespec to initialize to zero first and then manually update the appropriate fields. Also updates a raw syscall to use the libc function instead as on musl 1.2, it correctly handles libc::timespec values which, in musl 1.2, are always 16 bytes in length regardless of platform.

See also https://github.com/rust-lang/libc/pull/2088

alexcrichton commented 1 year ago

Thanks!