change mac osx compiler in official branded beta+release builds from using "-j1" to "-j4".

RESOLVED FIXED

Status

defect
RESOLVED FIXED
8 years ago
6 years ago

People

(Reporter: joduinn, Assigned: lsblakk)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

1) From RelEng meeting today: we have already been using "-j4" for m-c/aurora/beta nightlies; this is a change for "beta release" and "release" builds only.

2) From RapidRelease/cadence meeting: release drivers verbally ok'd to land this in FF10.0b3.
Assignee: nobody → lsblakk
Attachment #585582 - Flags: review?(nrthomas)
Comment on attachment 585582 [details] [diff] [review]
add -j4 to mozconfigs for {macosx,macosx64}/mozilla-{release,beta}

These days you want browser/config/mozconfigs/macosx-universal/release in the mozilla-beta repo. We would also need to land the change in m-c and m-a, so that it propagates through for future betas. Don't think you need to do m-r, because the in-tree mozconfigs aren't there until the next merge.
Attachment #585582 - Flags: review?(nrthomas) → review-
Oh, and we can probably make the same change for linux32/linux64 too, even though they're not the long pole for releases. Windows will have to stay at -j1 until we're using pymake.

joduinn that the meeting only signed off on doing mac though.
Attachment #585626 - Flags: review?(nrthomas) → review+
Attachment #585626 - Flags: approval-mozilla-beta+
Should we also sync the configs to mozilla-aurora and mozilla-central repos to prevent conflicts in the future?
Will this be gone from mozilla-beta after mozilla-aurora -> mozilla-beta merge? If yes, we should land this patch on mozilla-central and mozilla-aurora as well.
Comment on attachment 585626 [details] [diff] [review]
in-tree mozconfigs for mozilla-beta linux{32,64} and macosx-universal

This doesn't affect mozilla-aurora and mozilla-central because we don't build releases off them. It simplifies m-c -> m-a -> m-b migration though.

Risk to taking this patch (and alternatives if risky): low
Attachment #585626 - Flags: approval-mozilla-aurora?
Comment on attachment 585626 [details] [diff] [review]
in-tree mozconfigs for mozilla-beta linux{32,64} and macosx-universal

[Triage Comment]
Approved for Aurora.
Attachment #585626 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment on attachment 585626 [details] [diff] [review]
in-tree mozconfigs for mozilla-beta linux{32,64} and macosx-universal

Landed on mozilla-aurora: http://hg.mozilla.org/releases/mozilla-aurora/rev/098475237afe
And mozilla-central: http://hg.mozilla.org/mozilla-central/rev/294b0fcf805b
Attachment #585626 - Flags: checked-in+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Looks like Alex cut the merge off at http://hg.mozilla.org/releases/mozilla-aurora/rev/43065851180a, which was prior to our landing, so this didn't make it to beta. I just landed it manually there: http://hg.mozilla.org/releases/mozilla-beta/rev/ac0ccdaf0fd8
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.