Thunderbird beta release source builder failing to run configure

VERIFIED FIXED

Status

defect
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: wsmwk, Assigned: aleth)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 1 obsolete attachment)

+++ This bug was initially created as a clone of Bug #1243448 +++

jlund investigated.
Turns out we can't limp through a beta without this for Thunderbird. 
Because if this we're missing builders AV, checksums, etc 
ref: https://www.lucidchart.com/documents/view/b733c8be-e607-445f-824a-f3353c287294

gaston	21:06 < gaston > i dont know the releng details but maybe c-b needs a 'dumb' mozconfig like https://bug1243448.bmoattachments.org/attachment.cgi?id=8714413
gaston	21:08 < gaston > in (wild guess) mail/config/mozconfigs/linux64/source ?

Callek	jlund: howd we fix it for firefox?
Callek	jlund: I thought we did source after-the-push for it?
jlund	I think nthomas ended up forcing a checksums push that triggered things downstreams. though the order there is important
jlund	we then fixed the source builder for real which triggered jobs again.
jlund	Callek: it's better to fix the source builder I think. can we just land and retag your source mozconfig. is that the only blocker?
Callek	`probably`
Callek	it needs to land in a mail/ dir at least
This is is a straight port and should land only on beta for now.
Attachment #8715474 - Flags: review?(rkent)
Assignee: nobody → aleth
Status: NEW → ASSIGNED
I suspect the equivalent of https://hg.mozilla.org/build/buildbot-configs/rev/8b8b715b6951 will also be needed.
Unlike https://hg.mozilla.org/build/buildbot-configs/rev/8b8b715b6951, the buildbot config already had a source_mozconfig entry here. I don't know if that was doing any important work which will break if it's replaced by this.
Attachment #8715476 - Flags: review?(jlund)
Attachment #8715474 - Attachment is obsolete: true
Attachment #8715474 - Flags: review?(rkent)
Attachment #8715474 - Attachment is obsolete: false
Attachment #8715474 - Flags: review?(rkent)
Forgot staging.
Attachment #8715478 - Flags: review?(jlund)
Attachment #8715474 - Attachment is obsolete: true
Attachment #8715474 - Flags: review?(rkent)
Attachment #8715476 - Attachment is obsolete: true
Attachment #8715476 - Flags: review?(jlund)
Attachment #8715474 - Attachment is obsolete: false
Attachment #8715474 - Flags: review?(rkent)
Adding Magnus in case he wants to comment on the configuration change.

I assume that the intention is to port this to beta? Is it beta-only, or on comm-central as well for now? I don't see that the FF equivalent has been added to mozilla-beta, or is that coming soon?
(In reply to Kent James (:rkent) from comment #5)
> I assume that the intention is to port this to beta? Is it beta-only, or on
> comm-central as well for now? I don't see that the FF equivalent has been
> added to mozilla-beta, or is that coming soon?

This has landed on m-b: https://hg.mozilla.org/releases/mozilla-beta/log/tip/browser/config/mozconfigs/linux64/source
(In reply to aleth [:aleth] from comment #6)
> (In reply to Kent James (:rkent) from comment #5)
> > I assume that the intention is to port this to beta? Is it beta-only, or on
> > comm-central as well for now? 

Since it's on m-c as well, comment 1 is wrong and we should land it on c-c too. What is unclear to me is which buildbot-config files should ultimately refer to it (e.g. thunderbird_*.py ?).
Comment on attachment 8715474 [details] [diff] [review]
Add 'dumb' source mozconfig (port Bug 1243448 - beta release source builder failing to run configure)

OK I think I understand this well enough to be comfortable adding this.
Attachment #8715474 - Flags: review?(rkent) → review+
https://hg.mozilla.org/comm-central/rev/72e817a0f26cbca44d06ab1936e935946f2d5083
Bug 1245605 - Add 'dumb' source mozconfig (port Bug 1243448 - beta release source builder failing to run configure). r=rkent a=me CLOSED TREE DONTBUILD
https://hg.mozilla.org/releases/comm-aurora/rev/e4b79af8a54b49d2e5142d135f4818cb44fa40a7
Bug 1245605 - Add 'dumb' source mozconfig (port Bug 1243448 - beta release source builder failing to run configure). r=rkent a=aleth
https://hg.mozilla.org/releases/comm-beta/rev/e2692503df3187aa87940cb34425828a2a593526
Bug 1245605 - Add 'dumb' source mozconfig (port Bug 1243448 - beta release source builder failing to run configure). r=rkent a=aleth
Comment on attachment 8715474 [details] [diff] [review]
Add 'dumb' source mozconfig (port Bug 1243448 - beta release source builder failing to run configure)

Review of attachment 8715474 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry for the late notice, but I think we also want:

http://mxr.mozilla.org/comm-central/source/mail/config/mozconfigs/linux32/debug#21

(the two client.py things) added, otherwise the tarball/checked-out-code won't include mozilla-*
Attachment #8715478 - Flags: review?(jlund) → review+
Comment on attachment 8715505 [details] [diff] [review]
Followup to add client.py to the source mozconfig

rs+ Callek over IRC.
Attachment #8715505 - Flags: review?(rkent) → review+
https://hg.mozilla.org/comm-central/rev/fa68d82ce36b192d6458e3ab321041fb768ed913
Bug 1245605 - Followup to add client.py to the source mozconfig. rs=Callek a=aleth CLOSED TREE DONTBUILD
https://hg.mozilla.org/build/buildbot-configs/rev/22f1e00d700ce1961ec8180dc9607ffd278631a3
Bug 1245605 - Use 'dumb' source mozconfig for TB beta builds following bug 123448. r=jlund
https://hg.mozilla.org/releases/comm-aurora/rev/979006bb04f213907ada6906d6bf189a9cf5255e
Bug 1245605 - Followup to add client.py to the source mozconfig. rs=Callek a=aleth
https://hg.mozilla.org/releases/comm-beta/rev/f0fc8eb8187ce6f2d618013ef5f1433f32c42b23
Bug 1245605 - Followup to add client.py to the source mozconfig. rs=Callek a=aleth
Blocks: 1245853
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.