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
Update Rust nightly #17149
Update Rust nightly #17149
Conversation
Heads up! This PR modifies the following files: |
@bors-servo r+ |
📌 Commit 9842227 has been approved by |
Update both Rust versions <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17149) <!-- Reviewable:end -->
💔 Test failed - linux-rel-wpt |
@jdm Does that ring a bell? |
No. |
@bors-servo try |
@bors-servo r- try |
@bors-servo retry |
Update both Rust versions <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17149) <!-- Reviewable:end -->
💔 Test failed - linux-rel-css |
@bors-servo retry #17166 |
Update both Rust versions <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17149) <!-- Reviewable:end -->
💔 Test failed - linux-rel-css |
@bors-servo retry #17166 |
Update both Rust versions <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17149) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
@bors-servo clean |
@bors-servo clean try- |
This is ok to land now, right? |
@SimonSapin I think you said we shouldn't raise minimum version to 1.17 or something like that? |
Firefox's mach currently checks for 1.15.1+: This is changed by filing a bug like https://bugzilla.mozilla.org/show_bug.cgi?id=1337153 and CCing @rillian. I don't know if there's a policy for how long we want to support a release before raising the requirement. |
Firefox's only policy is that release Firefox be build with stable Rust. Otherwise I've just tried to balance the interests of those wanting new features vs those wanting less churn in their dev environment. |
I removed the Rust stable commit given mozilla-central forces us to stay in the past. |
@bors-servo r+ |
📌 Commit 4c8a0f0 has been approved by |
Update Rust nightly <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17149) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
This change is