Teach flight.mk about Camino-with-toolkit.jar

RESOLVED FIXED in mozilla1.9.3a4

Status

()

Core
Build Config
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Smokey Ardisson (offline for a while; not following bugs - do not email), Assigned: Smokey Ardisson (offline for a while; not following bugs - do not email))

Tracking

Trunk
mozilla1.9.3a4
All
Mac OS X
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(status1.9.2 .4-fixed, status1.9.1 .10-fixed)

Details

Attachments

(1 attachment)

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.
Attachment #432203 - Flags: review?(ted.mielczarek)
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.
Attachment #432203 - Flags: review?(ted.mielczarek) → review+
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 :-)
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
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.
Attachment #432203 - Flags: approval1.9.2.3?
Attachment #432203 - Flags: approval1.9.1.10?
Flags: in-testsuite-
Target Milestone: --- → mozilla1.9.3a4
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 ;-)
Attachment #432203 - Flags: approval1.9.2.3?
Attachment #432203 - Flags: approval1.9.1.10?
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 approval1.9.1.10+ 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 approval1.9.2.3+"
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.
Attachment #432203 - Flags: approval1.9.2.3?
Attachment #432203 - Flags: approval1.9.1.10?
(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.
Attachment #432203 - Flags: approval1.9.2.3?
Attachment #432203 - Flags: approval1.9.2.3+
Attachment #432203 - Flags: approval1.9.1.10?
Attachment #432203 - Flags: approval1.9.1.10+
Comment on attachment 432203 [details] [diff] [review]
Teach fix-buildconfig about Camino-with-toolkit.jar

a=beltzner for 1.9.1.10 and 1.9.2.3
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 :)
status1.9.1: --- → .10-fixed
status1.9.2: --- → .3-fixed
You need to log in before you can comment on or make changes to this bug.