-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Remove bitrig support from rust #60775
Conversation
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Ping @joshtriplett . Are you having time for this or should I choose another reviewer? |
On May 14, 2019 1:50:35 AM PDT, Marcel Hellwig ***@***.***> wrote:
Ping @joshtriplett . Are you having time for this or should I choose
another reviewer?
Looks generally reasonable to me. r=me if it passes CI. Currently traveling, though, so feel free to pick another reviewer.
|
Happy traveling then :) @bors r=joshtriplett |
@hellow554: 🔑 Insufficient privileges: Not in reviewers |
@bors r=joshtriplett |
📌 Commit cc314b0 has been approved by |
@bors rollup=never |
Remove bitrig support from rust Resolves #60743 using `find` and `rg` I delete every occurence of "bitrig" in the sources, expect for the llvm submodule (is this correct?). There's also this file https://github.com/rust-lang/rls/blob/5b8e99bb61958ca8abcb7c5eda70521726be1065/rls-analysis/test_data/rust-analysis/libstd-af9bacceee784405.json which contains a bitrig string in it. What to do with that?
☀️ Test successful - checks-travis, status-appveyor |
Bitrig support has been removed (by me) in rust-lang/rust#60775
bitrig support has been removed (by me) in rust-lang/rust#60775
bitrig support has been removed (by me) in rust-lang/rust#60775
Bitrig support was removed in rust-lang/rust#60775.
Bitrig support has been removed (by me) in rust-lang/rust#60775
Bitrig support was removed upstream in rust-lang/rust#60775. From the [merge commit][1] it landed in 1.36.0 which is long before our MSRV of Rust 1.56. [1]: rust-lang/rust@c84a7ab
Bitrig support was removed upstream in rust-lang/rust#60775. From the [merge commit][1] it landed in 1.36.0 which is long before our MSRV of Rust 1.56. This also fixes the `unexpected_cfgs` warning. [1]: rust-lang/rust@c84a7ab
Resolves #60743
using
find
andrg
I delete every occurence of "bitrig" in the sources, expect for the llvm submodule (is this correct?).There's also this file https://github.com/rust-lang/rls/blob/5b8e99bb61958ca8abcb7c5eda70521726be1065/rls-analysis/test_data/rust-analysis/libstd-af9bacceee784405.json which contains a bitrig string in it. What to do with that?