Closed Bug 712115 Opened 13 years ago Closed 13 years ago

L10n mac nightlies busted on central and aurora

Categories

(Release Engineering :: General, defect, P1)

x86
macOS
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: Pike, Assigned: armenzg)

References

Details

Attachments

(2 files)

Right now, the l10n nightlies are busted for mac, they can't find the stage dir.

Suspect would be bug 696436, as http://l10n.mozilla-community.org/~axel/nightlies/?date=2011-12-16 is green and http://l10n.mozilla-community.org/~axel/nightlies/?date=2011-12-17 is red.
So, yes, it seems like 696436 is the culprit.

The mozconfigs in the tree got an additional argument, 
ac_add_options --with-macbundlename-prefix=Firefox
https://hg.mozilla.org/releases/mozilla-aurora/diff/bd759f82e517/browser/config/mozconfigs/macosx-universal/nightly

Now, the l10n builds don't use a mozconfig but http://hg.mozilla.org/build/buildbotcustom/file/tip/process/factory.py#l3059.

Thus we need to fix this on the releng side :-/

Worse still, I have no idea how right now, as this needs to apply only for some branches, AFAICT?

Gavin, can you lay out what really needs to be impacted here?
Component: Build Config → Release Engineering
Product: Firefox → mozilla.org
QA Contact: build.config → release
Version: 6 Branch → other
Nightly and Aurora nightly mozconfigs on Mac now have --with-bundlename-prefix=Firefox specified, which modifies the value of MOZ_MACBUNDLE_NAME. I guess the easy fix would be to just add the same flag to the l10n pseudo-configs... It doesn't have any effect on non-Mac, so you could probably just add it everywhere.

Having these configs not be in sync is probably going to be a problem in the long-term, of course.
Assignee: nobody → armenzg
Comment on attachment 583282 [details] [diff] [review]
add --with-bundlename-prefix=Firefox to m-c and m-a

Per IRC, I don't like the naming, but r=me if we name this l10n_extra_configure_args.
Attachment #583282 - Flags: review?(aki) → review+
Comment on attachment 583286 [details] [diff] [review]
accept extraConfigureArgs variable and use it for L10n nightly builds

Same here.
Attachment #583286 - Flags: review?(aki) → review+
Comment on attachment 583282 [details] [diff] [review]
add --with-bundlename-prefix=Firefox to m-c and m-a

http://hg.mozilla.org/build/buildbotcustom/rev/12a44c7cc07e
Attachment #583282 - Flags: checked-in+
Comment on attachment 583286 [details] [diff] [review]
accept extraConfigureArgs variable and use it for L10n nightly builds

http://hg.mozilla.org/build/buildbot-configs/rev/e122370afc64
Attachment #583286 - Flags: checked-in+
This made it to production tonight.
Priority: -- → P1
I see some of the jobs going red.
I will clobber and see how it continues.
I see tons of green.

This is working now:
http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla-l10n-bn-IN/1324481280.1324481998.7149.gz&fulltext=1
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Did you get any updates today?

Please file a separate bug if you still do.
Works again, thanks.
Status: RESOLVED → VERIFIED
Blocks: 714284
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: