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

Link to rustc_driver crate in plugin #3564

Merged
merged 1 commit into from
Dec 20, 2018

Conversation

alexcrichton
Copy link
Member

This is in anticipation for rust-lang/rust#56987 where the
rustc_driver crate being linked in will be required to link correctly
against the compiler. In the meantime it should be harmless otherwise!

@matthiaskrgr
Copy link
Member

Looks like this needs reformatting

+++cargo +nightly fmt --all -- --check
Diff in /home/travis/build/rust-lang/rust-clippy/src/lib.rs at line 16:
 // FIXME: switch to something more ergonomic here, once available.
 // (currently there is no way to opt into sysroot crates w/o `extern crate`)
 #[allow(unused_extern_crates)]
-extern crate rustc_plugin;
-#[allow(unused_extern_crates)]
 extern crate rustc_driver;
+#[allow(unused_extern_crates)]
+extern crate rustc_plugin;
 use self::rustc_plugin::Registry;
 
 #[plugin_registrar]

This is in anticipation for rust-lang/rust#56987 where the
`rustc_driver` crate being linked in will be required to link correctly
against the compiler. In the meantime it should be harmless otherwise!
@alexcrichton
Copy link
Member Author

Updated!

@phansch
Copy link
Member

phansch commented Dec 20, 2018

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 20, 2018

📌 Commit b5f6eb6 has been approved by phansch

@bors
Copy link
Collaborator

bors commented Dec 20, 2018

⌛ Testing commit b5f6eb6 with merge 80c07d4...

bors added a commit that referenced this pull request Dec 20, 2018
Link to `rustc_driver` crate in plugin

This is in anticipation for rust-lang/rust#56987 where the
`rustc_driver` crate being linked in will be required to link correctly
against the compiler. In the meantime it should be harmless otherwise!
@bors
Copy link
Collaborator

bors commented Dec 20, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: phansch
Pushing 80c07d4 to master...

@bors bors merged commit b5f6eb6 into rust-lang:master Dec 20, 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.

4 participants