Uh oh!
There was an error while loading. Please reload this page.
Try to make top-level Cargo.toml work without __CARGO_TEST_ROOT. - #56194
Conversation
rust-highfive
commented
Nov 24, 2018
r? @aturon (rust_highfive has picked a reviewer for you, use r? to override) |
eddyb
commented
Nov 24, 2018
bors
commented
Nov 24, 2018
📌 Commit 894fd10 has been approved by |
bors
commented
Nov 24, 2018
Try to make top-level Cargo.toml work without __CARGO_TEST_ROOT.
The last failures I noticed before adding the `__CARGO_TEST_ROOT` hack were in `src/test/run-make/thumb-none-{cortex-m,qemu}`, and had to do with the fact that the top-level `Cargo.toml` uses nightly Cargo features.
If that's the only problem, this PR should work, and we can figure out how to unbreak RLS.bors
commented
Nov 24, 2018
💔 Test failed - status-travis |
rust-highfive
commented
Nov 24, 2018
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
eddyb
commented
Nov 24, 2018
So it is indeed an error, not just a warning. |
alexcrichton
commented
Nov 25, 2018
r=me if this strategy passes tests! |
eddyb
commented
Nov 25, 2018
@alexcrichton It doesn't ("error: current package believes it's in a workspace when it's not"). What we need is to either have the default be not treating a package as being in a workspace just because it's nested directory-wise (and only e.g. produce a warning), or have some opt-out mechanism (like Maybe a way to say that the |
alexcrichton
commented
Nov 25, 2018
I believe as of recently cargo supports an |
eddyb
commented
Nov 25, 2018
Oh, heh, it was already used for something else, I just didn't notice! @bors r=alexcrichton |
bors
commented
Nov 25, 2018
📌 Commit 20461a22b1276da654fd4909847ee94ec459128f has been approved by |
bors
commented
Nov 25, 2018
⌛ Testing commit 20461a22b1276da654fd4909847ee94ec459128f with merge dbce8625f9d6a9528e14c6f0972f589dd39e63c2... |
bors
commented
Nov 25, 2018
💔 Test failed - status-travis |
rust-highfive
commented
Nov 25, 2018
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@alexcrichton Hmm, on the CI, the build dir is Alternatively, a tmp dir could be used for tescases that are also Cargo projects. |
eddyb
commented
Nov 25, 2018
@bors r=alexcrichton |
bors
commented
Nov 25, 2018
📌 Commit 6aa4eb9 has been approved by |
bors
commented
Nov 25, 2018
Try to make top-level Cargo.toml work without __CARGO_TEST_ROOT.
The last failures I noticed before adding the `__CARGO_TEST_ROOT` hack were in `src/test/run-make/thumb-none-{cortex-m,qemu}`, and had to do with the fact that the top-level `Cargo.toml` uses nightly Cargo features.
If that's the only problem, this PR should work, and we can figure out how to unbreak RLS.bors
commented
Nov 26, 2018
☀️ Test successful - status-appveyor, status-travis |
rust-highfive
commented
Nov 26, 2018
Tested on commit rust-lang/rust@c14ab13. Direct link to PR: <rust-lang/rust#56194> 🎉 rls on windows: test-fail → test-pass (cc @nrc@Xanewok, @rust-lang/infra). 🎉 rls on linux: test-fail → test-pass (cc @nrc@Xanewok, @rust-lang/infra).
The last failures I noticed before adding the
__CARGO_TEST_ROOThack were insrc/test/run-make/thumb-none-{cortex-m,qemu}, and had to do with the fact that the top-levelCargo.tomluses nightly Cargo features.If that's the only problem, this PR should work, and we can figure out how to unbreak RLS.