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

rustbuild: Fix a few rebuilding issues #38809

Merged
merged 2 commits into from
Jan 4, 2017

Conversation

alexcrichton
Copy link
Member

Did a bit of investigation and found a few small unrelated issues, but this should help clean up a lot of errors we've been seeing locally.

Despite what the comment says, we actually need to do this. We're not cleaning
out the stage0 compiler's sysroot, but rather just our own sysroot that we
assembled previously.
Recent versions of Cargo lift less output up into the "main" directory, so let's
look more inside the `deps` folder for changes to propagate differences.

Closes rust-lang#38744
Closes rust-lang#38746
@rust-highfive
Copy link
Collaborator

r? @brson

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

@est31
Copy link
Member

est31 commented Jan 3, 2017

ref #38744 , #38746

many thanks!!

@brson
Copy link
Contributor

brson commented Jan 3, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Jan 3, 2017

📌 Commit 3ab778b has been approved by brson

@alexcrichton
Copy link
Member Author

@bors: p=1

@bors
Copy link
Contributor

bors commented Jan 4, 2017

⌛ Testing commit 3ab778b with merge a68622c...

bors added a commit that referenced this pull request Jan 4, 2017
rustbuild: Fix a few rebuilding issues

Did a bit of investigation and found a few small unrelated issues, but this should help clean up a lot of errors we've been seeing locally.
@bors
Copy link
Contributor

bors commented Jan 4, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing a68622c to master...

@bors bors merged commit 3ab778b into rust-lang:master Jan 4, 2017
@alexcrichton alexcrichton deleted the rustbuild-fresh branch January 13, 2017 23:27
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