Phil Ringnalda (:philor) Reporter
	Description • 10 years ago

You're going to love this. We only run Xulrunner as nightly builds, only on mozilla-central and mozilla-aurora, and hidden so that absolutely nobody ever looks at them, but, they are release deliverables so their failure is the very definition of a blocker.

https://tbpl.mozilla.org/php/getParsedLog.php?id=47125117&tree=Mozilla-Aurora is the latest Mac xulrunner build on mozilla-aurora, dying with the same dangerous destructor it has been dying with since the merge to mozilla-aurora on July 21st, and had been dying with on mozilla-central since July 7th (though trunk currently dies with a different failure, because we heard you liked bustage on your bustage).

In two days, aurora will merge to beta, we'll build a beta release, and one of the things that the beta release requires is a Mac xulrunner build, which ain't gonna build.

[Tracking Requested - why for this release]:
It's a release deliverable, until the long awaited bug 672509 finally gets rid of this insane "hidden tier-2 build is a release deliverable" situation.

	Benoit Jacob [:bjacob] (mostly away) Assignee
	Comment 1 • 10 years ago

Not sure how "nobody realized until now" is compatible with "this matters enough to block release" :-)

	Phil Ringnalda (:philor) Reporter
	Comment 2 • 10 years ago

Because that's how we do it: we release it, but the owner doesn't want it to be a tier-1 build which runs on-push on every tree so it's hidden and nightly-only, and the entire project lusts to run so goddamn many broken things that the single person who ever once looked at hidden jobs, me, finally got sick of looking at them and stopped. You're just lucky I gave you two days from having looked at hidden for something else, the way of the future is going to be "releng does a merge, builds a beta, retriggers the busted xulrunner build a few times, and then says 'huh, that's been broken for a couple of months, and we need it fixed tonight so we can release this beta.'"

	Benoit Jacob [:bjacob] (mostly away) Assignee
	Comment 3 • 10 years ago

I'll see what I can do today, but this is a holiday (as it is for you) and I have lots of other things to take care of. No promises. This should be super easy though, once I get to reproduce locally --- just found Xulrunner build instructions at https://developer.mozilla.org/en-US/docs/Mozilla/Projects/XULRunner/Build_Instructions

	Benoit Jacob [:bjacob] (mostly away) Assignee
	Comment 4 • 10 years ago

Reproduced locally, and the trivial 1-line fix works, so I landed it on inbound, without review, on a CLOSED TREE:

https://hg.mozilla.org/integration/mozilla-inbound/rev/b6832df4a1c1

Let me know if I should land it on other trees as well.

	Benoit Jacob [:bjacob] (mostly away) Assignee
	Comment 5 • 10 years ago

Philor/Ryan/Ed: I'm defaulting to assuming that I should leave it for you guys to backport an unreviewed fix to any other trees than inbound...

	Ed Morley [:emorley]
	Comment 6 • 10 years ago

(In reply to Benoit Jacob [:bjacob] from comment #5)
> Philor/Ryan/Ed: I'm defaulting to assuming that I should leave it for you
> guys to backport an unreviewed fix to any other trees than inbound...

So ordinarily you'd require a patch with the relevant approval+ given, but this is only a xulrunner/* file change so can land with a=npotdb.

I believe these changes should get it on the necessary bug queries (I don't have those repos checked out at the moment, and beta is closed for uplift still).

	Phil Ringnalda (:philor) Reporter
	Comment 7 • 10 years ago

Not beta, not unless it missed the merge - 32 is unaffected, if it was affected we'd still be trying to build beta1.

Strictly speaking it's not fixed on the trunk, since it hasn't merged from m-i yet, but then, there's bustage that kills the build before this bustage on the trunk so we aren't going to see whether or not this actually fixed the problem on the trunk until after bug 1060995 gets fixed. Close enough.

https://hg.mozilla.org/releases/mozilla-aurora/rev/1d9cbbf6af8c

	Phil Ringnalda (:philor) Reporter
	Comment 8 • 10 years ago

Green on aurora :)

	Phil Ringnalda (:philor) Reporter
	Comment 9 • 10 years ago

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