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

A script to conditionally install (or reinstall) clippy #2463

Closed
azriel91 opened this issue Feb 15, 2018 · 2 comments
Closed

A script to conditionally install (or reinstall) clippy #2463

azriel91 opened this issue Feb 15, 2018 · 2 comments

Comments

@azriel91
Copy link

Hiya, related to #2108, I wrote a script to

  • Install clippy if it is not already installed
  • If it is installed, check compatibility with current nightly. If it is compatible, do nothing.
  • If it is not compatible install it with --force.

Script: https://gist.github.com/azriel91/6d7ab1d865d266d50dabfc8da78f6490

Do you think this repo is where that script should live?

Oh, I've only tested it on Ubuntu, so if we were to combine this with the rust-update script, it would need to be adapted for Windows and OSX (I don't have one, so can't test).


Part of me is waiting for clippy to be released as a Rustup component: 👀

@TheIronBorn
Copy link

There is already a similar script https://github.com/rust-lang-nursery/rust-clippy#updating-rustc. Perhaps there are improvements to be made?

@phansch
Copy link
Member

phansch commented Dec 2, 2018

Thanks for the report and the script. Since we now ship Clippy only via rustup and only if it works, I'm going ahead and close this issue.

@phansch phansch closed this as completed Dec 2, 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

No branches or pull requests

3 participants