Closed Bug 1716735 Opened 3 years ago Closed 3 years ago

Move linux64-bp-clang-trunk from m-r to m-b in order to make it easier for uplift patches when build issues arise.

Categories

(Firefox Build System :: Toolchains, enhancement)

Product:
Firefox Build System
Component:
Toolchains
Type:
enhancement

Tracking

(firefox89 fixed, firefox90 fixed, firefox91 fixed)

Status:
RESOLVED FIXED
Milestone:
91 Branch
Tracking Flags:
Tracking Status
firefox89 --- fixed
firefox90 --- fixed
firefox91 --- fixed

People

(Reporter: andi, Assigned: andi)

References

Details

Attachments

(1 file)

Bug 1716735 - move linux64-bp-clang-trunk from m-r to m-b in order to make it easier for uplift patches when build issues arise. r=marco
48 bytes, text/x-phabricator-request
Details | Review

As was discussed in Bug 1707096 release managers are not very happy, and with good reason, to uplift patches to release that don't fix critical issues, so we should run this on m-b.

Summary: Move linux64-bp-clang-trunk from m-r to m-b in order to make it easier for uplift patches when build issues arise. r=marco → Move linux64-bp-clang-trunk from m-r to m-b in order to make it easier for uplift patches when build issues arise.
See Also: → 1707096
Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f0ac42dff458
move linux64-bp-clang-trunk from m-r to m-b in order to make it easier for uplift patches when build issues arise. r=marco

https://hg.mozilla.org/mozilla-central/rev/f0ac42dff458

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox91: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch

Presumably we can uplift this to release (89) at this point to stop running the failing build there?

Flags: needinfo?(bpostelnicu)

Comment on attachment 9227351 [details]
Bug 1716735 - move linux64-bp-clang-trunk from m-r to m-b in order to make it easier for uplift patches when build issues arise. r=marco

Beta/Release Uplift Approval Request

  • User impact if declined: none, only local infra
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Avoid running failed task of building ff with clang-trunk
  • String changes made/needed:
Flags: needinfo?(bpostelnicu)
Attachment #9227351 - Flags: approval-mozilla-release?

(In reply to Julien Cristau [:jcristau] from comment #5)

Presumably we can uplift this to release (89) at this point to stop running the failing build there?

Yes please!

Comment on attachment 9227351 [details]
Bug 1716735 - move linux64-bp-clang-trunk from m-r to m-b in order to make it easier for uplift patches when build issues arise. r=marco

uplifted as not part of the build

Attachment #9227351 - Flags: approval-mozilla-release?

Thanks Andi :)

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

Attachment

General

Creator:
Created:
Updated:
Size: