Closed Bug 445708 Opened 14 years ago Closed 14 years ago

Disable official branding on Thunderbird l10n tinderboxes.

Categories

(Thunderbird :: Build Config, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3

People

(Reporter: standard8, Assigned: standard8)

Details

Attachments

(2 files)

Attached patch The fixSplinter Review
Now we have unofficial branding working correctly on Thunderbird en-US nightlies, we should do the same on the l10n nightly builds (FF already ships as unofficial).

Attaching patch, not really sure who to get review from, so I'll check Alex is happy with this - basically I'm just porting the tinderbox bits I did in bug 397621/bug 443398 to the l10n branch in cvs.
Attachment #329984 - Flags: review?(l10n)
Comment on attachment 329984 [details] [diff] [review]
The fix

Sorry, I know jack about the interactions and back and forth of tinderbox configs, so I'm not a good reviewer for this.

I don't have a good replacement suggestion for this, either. Nick, Ben, Coop, any of those?
Attachment #329984 - Flags: review?(l10n) → review?
Attachment #329984 - Flags: review? → review?(nthomas)
Attachment #329984 - Flags: review?(nthomas) → review+
Current l10n tinderbox builds are broken due to missing branding files (ab-CD.jar!/locale/branding/*).

There will be only yellow error window:
XML Parsing Error: undefined entity
Location: chrome://messenger/content/messenger.xul
Line Number 18, Column 1:

(3.0a2pre, 2008071703, )
PS: if my analysis is right, this could be worked around by explicitly setting the branding to nightly.
I've just checked this in to trunk to fix the locale packaging problem. If it works, I'll get a review afterwards - my builds indicate that it will.

The problem here is that I decided to start putting the branding items into mail/branding/nightly, we can't just do --with-branding=mail/branding/nightly as that would mess up the windows installer packaging and a few other branding related items.

For the time being, in mail/locales/Makefile.in I've just specified the appropriate directory if MOZ_BRANDING_DIRECTORY is not defined.

Once we move to hg, I'll fix bug 443351 which will centralise all these unofficial branding items into one directory, and allow us to set MOZ_BRANDING_DIRECTORY permanently - hence cleaning up what we're doing in mail/locales/Makefile.in and elsewhere.
Comment on attachment 330198 [details] [diff] [review]
Fix locale packaging

This appears to work, I did a random sample of the hour builds (one on each platform) and they start with no problems for the branding.

Requesting review now it seems to work, please see comment 5 for the reasoning behind this patch.
Attachment #330198 - Flags: review?(bienvenu)
(In reply to comment #6)
> (From update of attachment 330198 [details] [diff] [review])
> This appears to work, I did a random sample of the hour builds (one on each
> platform) and they start with no problems for the branding.

Yes, this fixes the problem.
Comment on attachment 330198 [details] [diff] [review]
Fix locale packaging

oops, didn't see this request go by...
Attachment #330198 - Flags: review?(bienvenu) → review+
All reviews obtained, this is all checked in and I've heard about no further problems, so closing as fixed now.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3
You need to log in before you can comment on or make changes to this bug.