Skip to content
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

Clarify that .cargo/config files are unified #4485

Merged
merged 1 commit into from
Sep 13, 2017
Merged

Clarify that .cargo/config files are unified #4485

merged 1 commit into from
Sep 13, 2017

Conversation

carols10cents
Copy link
Member

It wasn't immediately clear to me from the text on this page whether Cargo looks for these files and stops at the first one, or whether it looks for them all and puts them all together. I was pretty sure it was the latter, but I think these few more words would have made me feel more confident sooner :)

@rust-highfive
Copy link

r? @alexcrichton

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

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Sep 12, 2017

📌 Commit 9b3685f has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Sep 12, 2017

⌛ Testing commit 9b3685f with merge 1cec9b5cd694bf38e134902cfe9e26cc798240a5...

@bors
Copy link
Collaborator

bors commented Sep 12, 2017

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member

alexcrichton commented Sep 13, 2017 via email

@bors
Copy link
Collaborator

bors commented Sep 13, 2017

⌛ Testing commit 9b3685f with merge 45d8fed...

bors added a commit that referenced this pull request Sep 13, 2017
Clarify that .cargo/config files are unified

It wasn't immediately clear to me from the text on this page whether Cargo looks for these files and stops at the first one, or whether it looks for them all and puts them all together. I was pretty sure it was the latter, but I think these few more words would have made me feel more confident sooner :)
@bors
Copy link
Collaborator

bors commented Sep 13, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 45d8fed to master...

@bors bors merged commit 9b3685f into rust-lang:master Sep 13, 2017
@ehuss ehuss added this to the 1.22.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.

5 participants