Closed Bug 851117 Opened 11 years ago Closed 11 years ago

Repack comm-aurora using in-tree mozconfigs, objdirs, and pre-built mar tools

Categories

(Thunderbird :: Build Config, defect, P2)

21 Branch
defect

Tracking

(thunderbird21 fixed)

RESOLVED FIXED
Tracking Status
thunderbird21 --- fixed

People

(Reporter: coop, Assigned: coop)

References

Details

Attachments

(1 file)

These changes are going live for Firefox and comm-central today, but comm-aurora repacks will break until the patch in https://bugzilla.mozilla.org/attachment.cgi?id=720094 is uplifted.

I'll re-attach the patch here and try to flag it appropriately to get some traction.
I'm trying to be conscientious and test the patch against comm-aurora. Unfortunately a straight uplift of my patch(es) from comm-central is failing, so trying to debug that. More info as I unravel it.
Status: NEW → ASSIGNED
(In reply to Chris Cooper [:coop] from comment #1)
> I'm trying to be conscientious and test the patch against comm-aurora.
> Unfortunately a straight uplift of my patch(es) from comm-central is
> failing, so trying to debug that. More info as I unravel it.

Turns out it was just aurora-specific path differences in how we need to set --with-l10n-base. Patch incoming.
[Approval Request Comment]
Regression caused by (bug #): bug 784848
User impact if declined: comm-aurora repacks will remain disabled
Testing completed (on c-c, etc.): c-aurora in releng dev env
Risk to taking this patch (and alternatives if risky): we're uplifting of some build config code prior to merge day, but this has already happened for mozilla-aurora.
Attachment #725139 - Flags: review?(mbanner)
Attachment #725139 - Flags: approval-comm-aurora?
Comment on attachment 725139 [details] [diff] [review]
comm-aurora changes required for repacking l10n on win64, using mozconfigs, objdirs, and pre-built mar tools

+ac_add_options --with-l10n-base=../../releases/l10n/mozilla-aurora

Just a thought (and could be done with a follow-up) does this dir need to have a repo specific name? If it was repo-independent it'd mean one less change to mozconfigs on merge day (although I automate it all for comm-central anyway, so not a huge issue).
Attachment #725139 - Flags: review?(mbanner)
Attachment #725139 - Flags: review+
Attachment #725139 - Flags: approval-comm-aurora?
Attachment #725139 - Flags: approval-comm-aurora+
Comment on attachment 725139 [details] [diff] [review]
comm-aurora changes required for repacking l10n on win64, using mozconfigs, objdirs, and pre-built mar tools

https://tbpl.mozilla.org/?tree=Thunderbird-Aurora&rev=ef46eb538736
(In reply to Mark Banner (:standard8) from comment #4) 
> +ac_add_options --with-l10n-base=../../releases/l10n/mozilla-aurora
> 
> Just a thought (and could be done with a follow-up) does this dir need to
> have a repo specific name? If it was repo-independent it'd mean one less
> change to mozconfigs on merge day (although I automate it all for
> comm-central anyway, so not a huge issue).

We want the same thing on the Firefox side. I'll file a follow-up.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.