Closed Bug 731043 Opened 10 years ago Closed 10 years ago

Builds from mozilla-beta shouldn't default to official branding


(Firefox Build System :: General, defect)

Not set


(firefox10+ fixed, firefox11+ fixed, firefox12 unaffected, firefox13 unaffected, firefox-esr1011+ verified)

Tracking Status
firefox10 + fixed
firefox11 + fixed
firefox12 --- unaffected
firefox13 --- unaffected
firefox-esr10 11+ verified


(Reporter: Dolske, Assigned: Gavin)


(Whiteboard: [qa!:esr10])


(2 files)

Earlier today a Gentoo developer dropped into #developers an grumbled about his build giving him official Firefox branding. I was all "lol ur build scripz are teh brok noob!" and shoo'd him away. A problem like that implies either the person building make some mistake, or we did something stupid. And since we _never_ do something stupid I spun up a build to check. Ahem.

Aaaanyway. My build from tip of hg:// with a minimal mozconfig (just enabling debug and setting an objdir) gives me a build with offical artwork. Sigh. This is a problem on our end and should be fixed. :(

This does raise the question of what the branding _should_ be. Ideally we'd have a nice generic "browser" branding, ala bug 682415. In the meantime we should revert back to using the previous "Aurora" branding (or even Nightly).
Just to be clear: this is about for the branding of builds people make themselves from the defaults. Official beta builds from Mozilla should have official branding, just as final releases do.
Looks like this was "broken" by:

which was in itself a bustage fix to a previous change made during the merge:

Short term fix is to just revert to using the Aurora or Nightly branding as the default for beta. The value isn't particularly important, since the release configs used for the builds shipped from the beta branch use the --enable-official-branding flag - it just affects what someone gets when they pull beta and build with a default config. Bug 659552 tracks fixing beta builds to have a suitable default branding that isn't confusing.
Attached patch patchSplinter Review
I went with "Nightly", since that's the default on trunk.
Assignee: nobody →
Attachment #601114 - Flags: review?(dolske)
Comment on attachment 601114 [details] [diff] [review]

"Anarchy" (original reporter of this bug) mentioned on IRC that mozilla-release is also affected. I think we'll want to land this there as well, just to avoid any future confusion (e.g. from Linux distros pulling mozilla-release code and generating their own builds). It has no impact on our release builds, since this is fallback that is overridden by our passing of --enable-official-branding.
Attachment #601114 - Flags: approval-mozilla-release?
Attachment #601114 - Flags: approval-mozilla-beta?
Attachment #601114 - Flags: review?(dolske) → review+
Attachment #601117 - Flags: review?(dolske) → review+
Comment on attachment 601114 [details] [diff] [review]

ESR is affected as well. Branch approval flag triagers: ping me on IRC if you need additional information. Just to recap:

- this doesn't affect our release builds in any way (they use the --enable-official-branding flag which overrides this default). There's no risk to our releases, but even if it did somehow manage to break something we'd notice it right away (the branding would be wrong).
- this affects people pulling code from these repos and building them with the default configuration (no --enable-official-branding flag specified). They're getting builds branded "Firefox" when they should be getting a build with unencumbered branding ("Nightly").
Attachment #601114 - Flags: approval-mozilla-esr10?
For background here, I was using a script provided to me for the merge process which apparently took a hammer to some of these files, thus causing the breakage in and the followup band-aid in
Comment on attachment 601114 [details] [diff] [review]

[Triage Comment]
Approving for release as well for those pulling/building off of m-r in the future, since this wouldn't affect the builds coming out of RelEng.
Attachment #601114 - Flags: approval-mozilla-release?
Attachment #601114 - Flags: approval-mozilla-release+
Attachment #601114 - Flags: approval-mozilla-esr10?
Attachment #601114 - Flags: approval-mozilla-esr10+
Attachment #601114 - Flags: approval-mozilla-beta?
Attachment #601114 - Flags: approval-mozilla-beta+

I also clobbered beta/release/ESR, given the change of branding.
Target Milestone: --- → Firefox 13
Closed: 10 years ago
Resolution: --- → FIXED
Verified that after the patch landed the branding for the mozilla-esr10 is the "Nightly" branding. 
Marking as Verified on firefox-esr10.
Whiteboard: [qa!:esr10]
Component: Build Config → General
Product: Firefox → Firefox Build System
Target Milestone: Firefox 13 → mozilla13
You need to log in before you can comment on or make changes to this bug.