search

mullvad / udp-over-tcp

Proxy UDP traffic over a TCP stream
Apache License 2.0
377 stars 61 forks source link

Make structopt optional #26

Closed faern closed 2 years ago

faern commented 2 years ago

I realize that structopt pulls in clap 2.x. And in our main repo we use clap 3.x.. So this will allow us to cut the dependency tree a bit.. Also, a library should not depend on a crate used to create CLIs...

Fixes #25

This change is Reviewable

faern commented 2 years ago

I realize we need to improve the CI. It's currently only doing cargo build && cargo test. It does not test with any of the features enabled.

faern commented 2 years ago

Seems to be a bug with cargo-all-features on beta and nightly. But it works. It just tries an unnecessary amount of feature combinations. https://github.com/frewsxcv/cargo-all-features/issues/27