Mozilla/5.0 (X11; Linux i686; rv:19.0) Gecko/19.0 Firefox/19.0 ID:20121021030553 Nightly builds for locales are branded as Firefox (Firefox menu button, "about Firefox menu"...) instead of Nightly. This is not profile related as I created new profiles to test. I tested French, Italian and English (US) builds, en-US builds are not affected.

I suspect that's the official-branding option in the l10n mozconfigs. See bugs 608004,558180

More likely due to bug 791209 which was deployed last week. Is the fix to remove "--enable-official-branding" from the mozconfigs?

I guess we need something similar to en-US, l10n-nightly and l10n-release or so.

Tacking on tracking 18 and 19 since this entirely blocks l10n stub installer testing and generally needs to be fixed.

(In reply to Jason Smith [:jsmith] from comment #5) > Tacking on tracking 18 and 19 since this entirely blocks l10n stub installer > testing and generally needs to be fixed. While we don't have a hard timeline for l10n stub installers, this is still a test blocker and completely breaks our Nightly branding for a large portion of our users. Can we take next steps here?

Comment on attachment 676584 [details] [diff] [review] remove official branding from l10n mozconfigs http://hg.mozilla.org/mozilla-central/rev/626b7ea149bc

Comment on attachment 676584 [details] [diff] [review] remove official branding from l10n mozconfigs I've just checked the size of all localized win32 builds and it bumped from ~200k to ~600k what is a good sign of the fact that the bug is fixed. See https://bugzilla.mozilla.org/show_bug.cgi?id=806280#c4 why. We need to uplift this to m-a. Bug caused by (feature/regressing bug #): User impact if declined: localized builds have wrong branding Testing completed (on m-c, etc.): fixed on m-c Risk to taking this patch (and alternatives if risky): low String or UUID changes made by this patch: None

Is this RESOLVED FIXED for trunk since there's a check in or is there more work to be done here? Just want to know if something is ready for verification.

Comment on attachment 676584 [details] [diff] [review] remove official branding from l10n mozconfigs [Triage Comment] Approving in support of l10n stub installer testing ahead of the 18->Beta merge.

With apologies for not reading all the bug ... is there any action needed when we next merge form Aurora to Beta, or are the mozconfigs modified only used for nightly repacks ?

Comment on attachment 676584 [details] [diff] [review] remove official branding from l10n mozconfigs Review of attachment 676584 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/config/mozconfigs/macosx-universal/l10n-mozconfig @@ +6,1 @@ > fi I'm seeing errors in tinderbox logs about this not being valid syntax, i.e., the empty command list after then. Guess we should completely remove the conditional, or comment it out. Not sure why some builds still pass, but bld-lion-r5-053 consistently fails on that error.