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

Revert "Merge pull request #3257 from o01eg/remove-sysroot" #3546

Merged
merged 2 commits into from
Dec 19, 2018

Conversation

matthiaskrgr
Copy link
Member

This reverts commit 041c49c, reversing
changes made to 1df5766.

The PR broke running a cargo-install'd clippy.
The installed clippy would not be able to find a crate for std.

Fixes #3523
Reopens #2874

@oli-obk
Copy link
Contributor

oli-obk commented Dec 14, 2018

I'd rather not revert but find a way to make it work. Can we implement this in cargo-clippy instead of clippy-driver?

@matthiaskrgr
Copy link
Member Author

Well I'd still like to have this reverted until an actual fix is established since this might break CI for people who install the git version on travois (which we advertised in README.md).

@oli-obk
Copy link
Contributor

oli-obk commented Dec 14, 2018

which we advertised in README.md

We did, we don't anymore. Using the git version is suboptimal for everyone as you have nearly no chance of figuring out the correct version.

We can add instructions for compiling clippy and manually copying the binaries into the correct location so it works again.

@matthiaskrgr
Copy link
Member Author

I can't even call the clippy binary manually after compiling it (like ./target/debug/cargo-clippy), it's hard to believe that this slipped through CI...

Can we apply the revert until the bug has been fixed?

@oli-obk
Copy link
Contributor

oli-obk commented Dec 19, 2018

@bors r+

We should focus on rust-lang/cargo#6356 to solve this correctly

@bors
Copy link
Collaborator

bors commented Dec 19, 2018

📌 Commit 9fb8022 has been approved by oli-obk

@bors
Copy link
Collaborator

bors commented Dec 19, 2018

⌛ Testing commit 9fb8022 with merge 980bcd8...

bors added a commit that referenced this pull request Dec 19, 2018
Revert "Merge pull request #3257 from o01eg/remove-sysroot"

This reverts commit 041c49c, reversing
changes made to 1df5766.

The PR broke running a cargo-install'd clippy.
The installed clippy would not be able to find a crate for std.

Fixes #3523
Reopens #2874
@bors
Copy link
Collaborator

bors commented Dec 19, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: oli-obk
Pushing 980bcd8 to master...

@bors bors merged commit 9fb8022 into rust-lang:master Dec 19, 2018
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.

3 participants