Closed Bug 1700562 Opened 4 years ago Closed 4 years ago

Update builders to rustc 1.51

Categories

(Firefox Build System :: Toolchains, task)

task

Tracking

(firefox89 fixed)

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: hsivonen, Assigned: glandium)

References

(Blocks 1 open bug)

Details

(Keywords: perf-alert)

Attachments

(3 files)

When doing this, please remove the special case for xmldecl from RUSTC_BOOTSTRAP and remove the git override for xmldecl (version 0.1.1 from crates.io works with Rust 1.51.)

The whole point of the special case is to keep it building with versions older than 1.51.

(In reply to Mike Hommey [:glandium] from comment #1)

The whole point of the special case is to keep it building with versions older than 1.51.

Our policy says "We require a new stable Rust starting 2 weeks after it is released.", so in two weeks, we don't need to support building with versions older than 1.51.

The policy is outdated. Our MSRV is currently 1.47 and will stay there until new code needs a newer version. Bug 1695828 is open about updating the policy document.

Assignee: nobody → mh+mozilla
Status: NEW → ASSIGNED
Depends on: 1701418
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/4dc85d4d7f33 Update builders to rustc 1.51. r=firefox-build-system-reviewers,andi,mhentges

Backed out for causing wrench bustages.

Backout link:https://hg.mozilla.org/integration/autoland/rev/647f1d665b2d88c8b84ecb7e9f1bae2315f7598c

Push with failures Wrench

Failure log Wrench

Also bustages in win 2012 debug

Push with failure win 2012

Failure log win 2012

error: unnecessary trailing semicolon
[task 2021-03-30T09:01:26.939Z] --> wrench\src\egl.rs:470:6
[task 2021-03-30T09:01:26.939Z] |
[task 2021-03-30T09:01:26.939Z] 470 | };
[task 2021-03-30T09:01:26.939Z] | ^ help: remove this semicolon
[task 2021-03-30T09:01:26.939Z] |
[task 2021-03-30T09:01:26.939Z] = note: -D redundant-semicolons implied by -D warnings
[task 2021-03-30T09:01:26.939Z]
[task 2021-03-30T09:01:28.115Z] error: aborting due to previous error
[task 2021-03-30T09:01:28.115Z]
[task 2021-03-30T09:01:28.206Z] error: could not compile wrench
[task 2021-03-30T09:01:28.206Z]
[task 2021-03-30T09:01:28.206Z] Caused by:
[task 2021-03-30T09:01:28.206Z] process didn't exit successfully: `rustc --crate-name wrench --edition=2018 wrench\src\main.rs --error-format=jso

Flags: needinfo?(mh+mozilla)
Flags: needinfo?(mh+mozilla) → needinfo?(kats)

Patch attached. I didn't test it since it seems kinda trivial.

Flags: needinfo?(kats)
Pushed by kgupta@mozilla.staktrace.com: https://hg.mozilla.org/integration/autoland/rev/8325da45ffd5 Remove stray semicolon in windows-only rust file. r=gfx-reviewers,bradwerth https://hg.mozilla.org/integration/autoland/rev/a61fef9be52b Update builders to rustc 1.51. r=firefox-build-system-reviewers,andi,mhentges
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/ce2e93d12fee Ensure we pass string literals to panic's first argument. r=glandium
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

== Change summary for alert #29539 (as of Thu, 01 Apr 2021 11:20:45 GMT) ==

Improvements:

Ratio Suite Test Platform Options Absolute values (old vs new)
8% perf_reftest_singletons bloom-basic-2.html linux1804-64-shippable e10s stylo 50.66 -> 46.72
7% perf_reftest_singletons bloom-basic.html linux1804-64-shippable e10s stylo 50.57 -> 46.86

For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=29539

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: