Closed Bug 1572836 Opened 1 year ago Closed 1 year ago

releasing off comm-esr68


(Thunderbird :: Build Config, defect)

Not set


(thunderbird_esr68 fixed)

Thunderbird 68.0
Tracking Status
thunderbird_esr68 --- fixed


(Reporter: rjl, Assigned: rjl)




(3 files)

Will use this bug to track build config issues related to doing the initial release of Thunderbird 68.0 off comm-esr68.

First up, build signing issues due to comm-esr68 not being recognized as a release branch.

Attaching so it can get into the THUNDERBIRD_68_VERBRANCH, will submit
to Phabricator for inclusion in M-C later.
Assignee: nobody → rob
Flags: needinfo?(jorgk)
See Also: → 1552389
This patch will not apply on c-c, as this particular change is already
present from work done in bug 1569539.

We may eventually uplift that bug, but it's pretty big and this is a relatively
safe change.

Try build successful with a beetmover upload:

Uploaded to:

Hmm, I think your talking about bug 1516816 and some of this:

Comment on attachment 9084854 [details] [diff] [review]

This fixes the missing linux64-en-US builds from 68.0-build2.
Scriptworker does not expand variables in artifact filenames so '${locale}' needs to be dropped.

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: 
Testing completed (on c-c, etc.): tested on try-comm-central, including the beetmove job that currently fails on comm-esr68
Risk to taking this patch (and alternatives if risky):
Minimal risk, the end result of this change is gdata-provider.en-US.xpi is now named gdata-provider.xpi. Geoff this particular change is in C-C already. It should be mentioned in the release notes in case a downstream packager relies on that filename.
Attachment #9084854 - Flags: review?(jorgk)
Attachment #9084854 - Flags: approval-comm-esr68?
Comment on attachment 9084854 [details] [diff] [review]

I'll push it now.
Attachment #9084854 - Flags: review?(jorgk) → review+
Attachment #9084854 - Flags: approval-comm-esr68? → approval-comm-esr68+

(In reply to Jorg K (GMT+2) from comment #4)

Hmm, I think your talking about bug 1516816 and some of this:

Yes, but that was only needed because of the changes in bug 1569539.

What are we going to do for beta when neither bug 1516816 nor bug 1569539 are going there? This hasn't been an issue on TB 69 beta for some reason. So we do nothing for that beta, right?

Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 68.0

Ahh. This got messy. The first part of bug 1516816 is on beta:, so it will run into the linux64-en-US bug.

Okay so I'm not completely crazy then, because I remember checking that I could use a variable there...

Beta can handle it, currently esr68 cannot.

This bug broke the tests. This patch should fix the tests. r+ from rjl on IRC.

Attachment #9090569 - Flags: review+
Attachment #9090569 - Flags: approval-comm-esr68?
Comment on attachment 9090569 [details] [diff] [review]
More gData fun :-(
Attachment #9090569 - Flags: approval-comm-esr68? → approval-comm-esr68+
You need to log in before you can comment on or make changes to this bug.