oconnor663 / os_pipe.rs

a cross-platform library for opening OS pipes in Rust
MIT License
100 stars 16 forks source link

Move src/bin/* to examples #18

Open ignatenkobrain opened 5 years ago

ignatenkobrain commented 5 years ago

Is there any reason to be able to do cargo install those binaries?

oconnor663 commented 5 years ago

Same question as in https://github.com/oconnor663/duct.rs/issues/73:

I should make the tests that use these integration tests rather than unit tests, so that the binaries get built automatically and we don't need the path_to_exe hack. Anyway, I was under the impression if you wanted tests to have access to binaries, they needed to be in src/bin/. Is that wrong?

loadenmb commented 5 years ago

Maybe switch command on windows / nix. Depends if system env data is available here. If not just execute "/usr/bin/sleep" if stderr it is a Windows system. For Windows we can execute: timeout /T 10 /nobreak It's to simple. So I'm not sure if I overlooked a important point.