Closed Bug 1564638 Opened 5 years ago Closed 5 years ago

Update Linux to Rust beta 1.37

Categories

(Firefox Build System :: General, task)

All
Windows
task
Not set
normal

Tracking

(firefox70 fixed)

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: erahm, Assigned: erahm)

References

Details

Attachments

(1 file, 1 obsolete file)

We need to update a platform to the latest rust beta 1.37 in order to enable rust PGO on Firefox nightly for testing. Lets just do Windows for now.

Depends on: 1564643
Depends on: 1564647
Summary: Update Windwos to Rust beta 1.37 → Update Windows to Rust beta 1.37

Windows still has a build issue upstream, lets just do Linux for now.

Summary: Update Windows to Rust beta 1.37 → Update Linux to Rust beta 1.37
Depends on: 1565359

Currently busted in the repackaging step, it looks like the install script for rust-src-beta has DOS line endings causing the interpreter line to be invalid.

With a hack to deal with the bogus line endings in repack_rust.py we've got functioning builds. We either need to land the workaround or wait for the fix on rust nightly to get backported to rust beta 1.37 and then for the next build to come along.

Depends on: 1565757
No longer depends on: 1565757

(In reply to Eric Rahm [:erahm] from comment #1)

Windows still has a build issue upstream, lets just do Linux for now.

There's a pending PR for a proper fix, that still needs to get reviewed and uplifted to rust beta 1.37 before we can enable on Windows.

(In reply to Eric Rahm [:erahm] from comment #3)

Currently busted in the repackaging step, it looks like the install script for rust-src-beta has DOS line endings causing the interpreter line to be invalid.

There's a fix landed in rust 1.38 nightly, but it hasn't been approved for uplift yet. I'm inclined to just add my workaround to our repackaging step.

DOS line endings ended up in some of rust's install.sh scripts. This adds a
step to proprecess the scripts to use proper line endings. We can remove this
once a proper fix is landed upstream and approved for backporting to rust
beta 1.37 or we can just keep this step in case the problem happens again.

Attachment #9078874 - Attachment is obsolete: true
Pushed by erahm@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2adf316da439
Update Linux to Rust beta 1.37. r=froydnj
Pushed by erahm@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a1a4f3976b34
Update Linux to Rust beta 1.37. r=froydnj,glandium
Attachment #9077519 - Attachment description: Bug 1564638 - Update Linux to Rust beta 1.37. r=glandium → Bug 1564638 - Update Linux to Rust beta 1.37. r=froydnj,glandium
Pushed by erahm@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/530d09741a0d
Update Linux to Rust beta 1.37. r=froydnj,glandium
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Assignee: nobody → erahm
Flags: needinfo?(erahm)
Blocks: 1571836
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: