search

ceph / ceph-rust

Rust-lang interface to Ceph.
Apache License 2.0
98 stars 43 forks source link

Support building on macOS and other unix platforms #41

Closed dae closed 5 years ago

dae commented 5 years ago

This makes it possible to check the code compiles without having to copy it across to a Linux machine.

In the future it may be worth removing these completely, or adding them to all of the crate's files - at the moment as only some parts of the code are flagged, other parts like cmd.rs spit out errors when on a non-supported platform.

dae commented 5 years ago

Sorry if I was not clear. The patch changes all the linux cfg entries to unix as far as I'm aware - I was just pointing out that even before this patch, not all code is wrapped in those conditional statements, which causes the build to fail when building on an unsupported platform.

ChrisMacNaughton commented 5 years ago

Ah, gotcha! Thanks for this change!