Skip to content

Defer bailing out workspace root probing#4816

Merged
bors merged 1 commit intorust-lang:masterfrom
alexcrichton:fix-workspace-is-home
Dec 13, 2017
Merged

Defer bailing out workspace root probing#4816
bors merged 1 commit intorust-lang:masterfrom
alexcrichton:fix-workspace-is-home

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

This commit alters the logic to bail out on probing for CARGO_HOME to
do it a little later rather than early on in the loop iteration, notably
allowing members to reside in CARGO_HOME itself.

Closes #4815

This commit alters the logic to bail out on probing for `CARGO_HOME` to
do it a little later rather than early on in the loop iteration, notably
allowing members to reside in `CARGO_HOME` itself.

Closes rust-lang#4815
@rust-highfive
Copy link
Copy Markdown

r? @matklad

(rust_highfive has picked a reviewer for you, use r? to override)

@matklad
Copy link
Copy Markdown
Contributor

matklad commented Dec 13, 2017

bors r+

@matklad
Copy link
Copy Markdown
Contributor

matklad commented Dec 13, 2017

@bors r+

@bors
Copy link
Copy Markdown
Contributor

bors commented Dec 13, 2017

📌 Commit 4f28bc1 has been approved by matklad

@bors
Copy link
Copy Markdown
Contributor

bors commented Dec 13, 2017

⌛ Testing commit 4f28bc1 with merge 9d05f7e...

bors added a commit that referenced this pull request Dec 13, 2017
Defer bailing out workspace root probing

This commit alters the logic to bail out on probing for `CARGO_HOME` to
do it a little later rather than early on in the loop iteration, notably
allowing members to reside in `CARGO_HOME` itself.

Closes #4815
@matklad
Copy link
Copy Markdown
Contributor

matklad commented Dec 13, 2017

Wow, it's really hard to fix little corner cases without breaking someone else's code =/

@bors
Copy link
Copy Markdown
Contributor

bors commented Dec 13, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 9d05f7e to master...

@bors bors merged commit 4f28bc1 into rust-lang:master Dec 13, 2017
@alexcrichton alexcrichton deleted the fix-workspace-is-home branch December 18, 2017 19:47
@ehuss ehuss added this to the 1.24.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nightly Rust using cargo 0.25.0-nightly (930f9d949 2017-12-05) fails to respect --frozen

5 participants