"unescaper" dependency is unnecessary when libudev is not used

[JonathonReinhart]

The unescaper crate is only used in one function, udev_property_encoded_or_replaced_as_string(), which is guarded by the libudev feature.

However, the unesecaper dependency is not conditional on the libudev feature: Cargo.toml. It is included for all linux builds.

[sirhcel]

Thank you @JonathonReinhart! Good catch.

To my understanding, enabling an optional dependency based on a feature of the crate is possible with the "dep:[...]" syntax since Rust 1.60 but not with our current MSRV 1.59. Do you know whether there is a way to get this working with 1.59?

What's the actual hassle you are experiencing with having this unused dependency on board?

[JonathonReinhart]

Do you know whether there is a way to get this working with 1.59?

I'm not sure, I'm not that familiar with Cargo's dependency features.

What's the actual hassle you are experiencing with having this unused dependency on board?

I'm considering importing the serialport crate into Android's vendored repository android-crates-io, which would also require importing the adds a false unescaper dependency. It's not a showstopper, it just an additional chore. requires some manual intervention with the tooling.

Though this isn't as big of a deal as the dependency on older nix (#175).