OpenBSD doesn't have lutimes so use utimensat instead.

alexcrichton / filetime

Accessing file timestamps in a platform-agnostic fashion in Rust

Apache License 2.0

122 stars 56 forks source link

OpenBSD doesn't have lutimes so use utimensat instead. #34

Closed ltratt closed 5 years ago

ltratt commented 5 years ago

Without this, compilation on OpenBSD causes the following error:

error[E0425]: cannot find value `lutimes` in module `libc`
  --> src/unix/utimes.rs:12:42
   |
12 |     super::utimes(p, atime, mtime, libc::lutimes)
   |                                          ^^^^^^^ did you mean `futimes`?

error: aborting due to previous error

which means that things like mdbook can't build.

ltratt commented 5 years ago

The errors don't appear to be related to this PR (SSL errors downloading rustup?!).

ltratt commented 5 years ago

Thanks for merging! At the risk of being cheeky, might a 0.2.3 release be on the cards? It would save us having to manually hack this patch in when compiling other crates that rely on filetime.

alexcrichton commented 5 years ago

Oh sure thing!

ltratt commented 5 years ago

Thanks Alex :)