Skip to content

Reintroduce extern crate for non-Cargo dependencies. #3189

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Sep 15, 2018

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Sep 15, 2018

See rust-lang/rust#53166 (comment) for the decision. Needed for rust-lang/rust#54116.

Includes a revert for #3160, to allow updating src/tools/clippy in rust-lang/rust.
(cc @matthiaskrgr)

@oli-obk
Copy link
Contributor

oli-obk commented Sep 15, 2018

Dogfood is failing, can you just allow the failing lint?

@eddyb
Copy link
Member Author

eddyb commented Sep 15, 2018

No, the lint was wrong. Also, it's very badly named IMO.

@eddyb
Copy link
Member Author

eddyb commented Sep 15, 2018

Just confirmed that this fixes ./x.py build src/tools/clippy for rust-lang/rust#54116.

@oli-obk oli-obk merged commit f166b7d into rust-lang:master Sep 15, 2018
@eddyb eddyb deleted the rextern branch September 15, 2018 08:37
eddyb added a commit to eddyb/rls that referenced this pull request Sep 15, 2018
eddyb added a commit to eddyb/rls that referenced this pull request Sep 15, 2018
eddyb added a commit to eddyb/rls that referenced this pull request Sep 15, 2018
eddyb added a commit to eddyb/rls that referenced this pull request Sep 15, 2018
eddyb added a commit to eddyb/rls that referenced this pull request Sep 15, 2018
eddyb added a commit to eddyb/rust that referenced this pull request Sep 15, 2018
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request May 5, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants