Closed Bug 1872891 Opened 2 years ago Closed 1 year ago

[comm-central] perma gmake[4]: *** [Makefile:38: vendored-rust-check] Error 1

Categories

(Thunderbird :: Upstream Synchronization, defect, P5)

Tracking

(thunderbird_esr128 unaffected)

RESOLVED FIXED
121 Branch
Tracking Status
thunderbird_esr128 --- unaffected

People

(Reporter: intermittent-bug-filer, Assigned: dandarnell)

References

(Regression)

Details

(Keywords: leave-open, regression)

Attachments

(4 files)

Summary: Intermittent gmake[4]: *** [Makefile:38: vendored-rust-check] Error 1 → [comm-central] Intermittent gmake[4]: *** [Makefile:38: vendored-rust-check] Error 1

Is there a point having them enabled, but perma busted? Why not disable until the builds are normally green?

Flags: needinfo?(rob)
Flags: needinfo?(daniel)
Summary: [comm-central] Intermittent gmake[4]: *** [Makefile:38: vendored-rust-check] Error 1 → [comm-central] perma gmake[4]: *** [Makefile:38: vendored-rust-check] Error 1

It appears that rust dependencies need to be refreshed. I'm working on a patch now.

Flags: needinfo?(daniel)

(In reply to Magnus Melin [:mkmelin] from comment #1)

Is there a point having them enabled, but perma busted? Why not disable until the builds are normally green?

Sort of...

Part of this is to see just how often we end up perma-busted, and then to test our tooling for effectiveness.

Running ./mach tb-rust vendor should fix the bustage. At least until m-c updates Rust dependencies again.

This really could have been done in the ash repo.

The path towards automation: A try job checks for deps out of sync and runs tb-rust vendor if needed. It can submit a patch to Phabricator (hopefully) and ping #thunderbird-ci.

This looks like:

$ mach tb-rust check-upstream
Rust dependencies are out of sync. Run `mach tb-vendor`.

Cargo.lock

Followed by a Python import error... I'll fix that.

$ mach tb-rust vendor
[ lots of output ]
$ diff -qr ../third_party/rust third_party/rust
Only in ../third_party/rust: autocfg
Only in ../third_party/rust: mio-0.6.23
Only in ../third_party/rust: rure
Only in ../third_party/rust: .sources
[ this looks correct ]
$ mach tb-rust check-upstream
Rust dependencies are okay.

Make sure to hg addremove . and commit, and include m-c hash in the commit message for reproducibility purposes.

Flags: needinfo?(rob)
Assignee: nobody → daniel
Status: NEW → ASSIGNED
Attachment #9371161 - Attachment description: Bug 1872891 - Update Rust dependencies. r=#thunderbird-build-system-reviewers → Bug 1872891 - Update Rust dependencies (mozilla-central: a32652cce394). r=#thunderbird-build-system-reviewers

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/e7d81463de9f
Update Rust dependencies (mozilla-central: a32652cce394). r=rjl

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED

For the moment, these will happen. To fix, run the steps in comment 3.

Status: RESOLVED → REOPENED
Type: task → defect
Component: Build Config → Upstream Synchronization
Resolution: FIXED → ---
Attachment #9371567 - Attachment description: Bug 1872891 ' Sync Rust code with mozilla-central. r=#thunderbird-reviewers → Bug 1872891 - Sync Rust code with mozilla-central. r=#thunderbird-reviewers
Pushed by john@thunderbird.net: https://hg.mozilla.org/comm-central/rev/150dd9e76b0c Sync Rust code with mozilla-central. r=john.bieling
Pushed by ikey@thunderbird.net: https://hg.mozilla.org/comm-central/rev/e958d43fcaa2 Update Rust dependencies (mozilla-central: 16becc119397). r=thunderbird-reviewers,babolivier
Pushed by ikey@thunderbird.net: https://hg.mozilla.org/comm-central/rev/d3fb50d2fa63 Update Rust dependencies (mozilla-central: f593f07c9772). r=babolivier DONTBUILD

Ever since bug 1869860 (and subsequent related bugs), this bug is no longer relevant, since Rust third-party vendoring is (semi-)automatic.

Status: REOPENED → RESOLVED
Closed: 2 years ago1 year ago
Resolution: --- → FIXED
See Also: → 1869860
Target Milestone: --- → 121 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: