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
Pass CARGO_HOME on more invocations of cargo #61194
Conversation
I lost the log, but I had an issue compiling cargo with it set to
|
Please cc the Nixpkgs maintainers of cargo/rust in order to get this further. |
I'm not sure who that is, but based on commit history, maybe it's @Mic92 or @Ericson2314 ? |
btw this may be fixed by rust-lang/cargo#7149 |
@@ -121,6 +122,7 @@ in stdenv.mkDerivation (args // { | |||
checkPhase = args.checkPhase or '' | |||
runHook preCheck | |||
echo "Running cargo test" | |||
export CARGO_HOME=$(pwd) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would rather set it to $TMPDIR
though. And it only needs to be set once - maybe in the configurePhase
?
With nightly 2019-08-06 built with beta 2019-08-06, I no longer have this issue. |
This should be covered by the following commit: 3372343 |
I think so too! |
Fixes #61192.
Motivation for this change
Although this failure only occurs when using Cargo nightly, which isn't actually packaged here yet, this will eventually become a problem when nightly becomes stable.
I'm not really sure that this is the right place for this change. Is the existing absence of
CARGO_HOME
meant to flush out reproducibility failures when cargo tries to download a package? Or was it just because previouslycargo
didn't need to do anything in the build or test phases so it wasn't necessary?Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)