search

capnproto / capnpc-rust

Cap'n Proto code generation for Rust
76 stars 26 forks source link

Hard-coded /bin/cat #9

Closed acertain closed 9 years ago

acertain commented 9 years ago

Not all systems have cat in /bin, specifically this fails on nixos.

dwrensha commented 9 years ago

Would it work for you if it got the absolute path of cat by running which cat?

P.S. I added a new "-o -" flag to capnp compile, so in the future we won't need cat https://github.com/sandstorm-io/capnproto/pull/190

acertain commented 9 years ago

Yes, that would work, for now I've edited the hard-coded path.

-o - in capnp sounds like the better solution

dwrensha commented 9 years ago

Addressed in https://github.com/dwrensha/capnpc-rust/commit/d2b81e040abd936fdaa70883965ccc850343c5ab.