-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OUT_DIR is not defined during cargo test
#879
Comments
Hm could you give an example of what's broken here? Some simple smoke tests work so I'm a little unclear on what's going wrong. |
Works for me on an executable crate but not on a library crate. E.g.:
|
Aha, I see what's going on here! It looks like it's just the doc tests that cause a problem. |
Did this fix make it into the pre-alpha versions? I'm getting this error when trying to build the "rust-http" module, just downloaded from "https://github.com/chris-morgan/rust-http/archive/master.zip".
|
Yes this change should definitely be in that version of Cargo. Perhaps a local bug? |
It's a problem with the build script for "rust-http". Just for the record, rust-http has a program which generates "read_method.rs", but into the codegen directory, which is not OUT_DIR. "rust-http" is now abandoned - ("OBSOLETION NOTICE...This library is DEAD...YOU SHOULD NOT USE THIS LIBRARY AT ALL.") So this will not be fixed. Switching to "Hyper" for HTTP use. Sigh. |
I think I'm having that same problem, when using OUT_DIR just in tests:
Resulting in:
I have a workaround: adding a
Resulting in:
|
@pbougue currently |
Ok thanks, we'll let it that way then! |
Oh wow, that wasn't very obvious. |
This makes it difficult to test libraries that depend on generated code.
The text was updated successfully, but these errors were encountered: