Open therealjasonkenney opened 1 year ago
Honestly, I don't see why we should wrap everything in platform checks, or provide stub implementations for effectively the entire API 🙂
This looks like a lot of work and extra build combinations on our side, whereas it's probably just a few lines on your side if it's just about logging.
I'm inclined to close this.
Fair, can you add something like this: https://stackoverflow.com/questions/72990789/how-to-specify-the-target-os-in-cargo-toml in lib.rs
?
It would make debugging for people who are using crates that chose to add your crate as a dependency much easier :)
I thought it'd be obvious that this crate only works on Linux, given that systemd itself only supports Linux 🙂
@swsnr I'm running into the same issue as well trying to use a different package that indirectly imports libsystemd
. I didn't know much about systemd, so it wasn't obvious to me.
I use OSX as a dev machine, and it would be nice if this could install just so I don't have wrap everything in platform checks. My project uses journald on linux and defaults to stderr when it's not available.
Anyhow here are the errors, and some seemed simple to fix