Created attachment 432203 [details] [diff] [review] Teach fix-buildconfig about Camino-with-toolkit.jar In order to create Universal Binaries of Camino-with-toolkit.jar, fix-buildconfig's variables in flight.mk need to reflect that Camino-with-toolkit.jar has toolkit.jar.
It's always a tough bet, figuring how someone will break you by not seeing you at some time in the future, but I'd think that just moving to a single assignment to BUILDCONFIG_JAR for everyone, right below where APP_CONTENTS is set, would be safer.
Gah, except you need to land on a branch or more, so your redundant but clearly NPOTB way is actually better.
Landed on trunk: http://hg.mozilla.org/mozilla-central/rev/36e430e9901a Phil, if you think moving stuff around there is a good idea, I'm fine with filing a follow-up, but as you noted and we discussed, this route is the easiest and therefore the best one for what we actually *need* for Camino :-)
Comment on attachment 432203 [details] [diff] [review] Teach fix-buildconfig about Camino-with-toolkit.jar Requesting branch approvals for this patch, which is needed for Universal Camino builds on 1.9.x>0. This change is inside a Camino-only ifdef and as such only affects Camino.
Comment on attachment 432203 [details] [diff] [review] Teach fix-buildconfig about Camino-with-toolkit.jar (In reply to comment #4) You don't need approval for NPOTB changes ;-)
Serge, don't cancel my flag requests; you're not the patch author, module owner or peer, or a branch driver. Further, to quote the tinderboxen rules: "Bugs must have approval220.127.116.11+ from the 1.9.1-branch triage team (or not touch files shared by Firefox or XULRunner and have branch approval from the owners of that part of the code)" "The only patches permitted to land on mozilla-1.9.2 are patches that have explicit approval18.104.22.168+"
Comment on attachment 432203 [details] [diff] [review] Teach fix-buildconfig about Camino-with-toolkit.jar Drivers, please see comment 4 for the full explanation from when I originally requested approval before the request was erroneous cancelled out from from under me (but, in short, Camino needs this fix to be able to ship, and the change is inside a Camino-only ifdef, so the change will only affect Camino). Thanks.
(In reply to comment #6) I know the tinderbox rules, yet my comment 5 stands: that's how we do for OS/2 for example. But if you insist on requesting approval, fine with me.
Comment on attachment 432203 [details] [diff] [review] Teach fix-buildconfig about Camino-with-toolkit.jar a=beltzner for 22.214.171.124 and 126.96.36.199
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/c102cbe300db http://hg.mozilla.org/releases/mozilla-1.9.1/rev/8f0e089e1a05 Note to QA: this is a build-only fix that can only be verified by the successful creation of Camino nightlies (or by code inspection); there aren't currently Camino nightlies on either of these branches, but I do hope to have 1.9.2 nightlies by the end of the week now that this bug is in :)