Closed Bug 716842 Opened 12 years ago Closed 12 years ago

l10n repacks need the makefile in mobile/locales


(Firefox for Android Graveyard :: General, defect)

Not set


(firefox11 fixed)

Firefox 12
Tracking Status
firefox11 --- fixed


(Reporter: Pike, Assigned: Pike)




(1 file)

Missed this when doing bug 708015, the Makefile in mobile/locales is currently only done when building from mobile/android or /xul, per DIRS.

It's not there when you start your journey in mobile/android/locales.

Do what branding does is the way to go.
This makes sure that the mobile/locales/Makefile is created, by both configure and the Makefile logic in mobile/*/locales/

Trivial patch, also pushed to try at with a minimal build set.
Attachment #587270 - Flags: review?(aki)
Hm, I'm getting

 extracting: omni.ja                 
  inflating: classes.dex             
mv: cannot stat `lib/armeabi-v7a/': No such file or directory
make: *** [/builds/slave/m-cen-andrd-l10n-1/mozilla-central/obj-l10n/mobile/android/locales/../../../dist/l10n-stage/fennec] Error 1
command: END (0.39s elapsed)

Traceback (most recent call last):
  File "/builds/slave/m-cen-andrd-l10n-1/scripts/scripts/l10n/", line 191, in <module>
    options.platform, stage_platform, mobileDirName, en_us_binary_url)
  File "/builds/slave/m-cen-andrd-l10n-1/scripts/scripts/l10n/", line 51, in createRepacks
    buildInfo = downloadNightlyBuild(localeSrcDir, env)
  File "/builds/slave/m-cen-andrd-l10n-1/scripts/scripts/l10n/../../lib/python/build/", line 8, in downloadNightlyBuild
    run_cmd(["make", "unpack"], cwd=localeSrcDir, env=env)
  File "/builds/slave/m-cen-andrd-l10n-1/scripts/scripts/l10n/../../lib/python/util/", line 42, in run_cmd
    return subprocess.check_call(cmd, **kwargs)
  File "/tools/python-2.5.1/lib/python2.5/", line 461, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['make', 'unpack']' returned non-zero exit status 2

I currently suspect .
[11:11]	<glandium>	aki: it suggests you're using a that is newer than the build you're using it on

Building a new nightly to test on.
Yeah, looks OK as long as your build and your tree are on the same side of the tree.
Comment on attachment 587270 [details] [diff] [review]
generate mobile/locales/Makefile

I have android l10n jobs going green now (with this patch), thanks!
There are some that are broken, but those look like locales that need updating (

    raise RuntimeError('File "%s" not found in %s' % (src, ', '.join(src_base)))
RuntimeError: File "wikipedia-he.xml" not found in /builds/slave/m-cen-andrd-l10n-3/mozilla-central/mobile/locales/en-US/searchplugins, /builds/slave/m-cen-andrd-l10n-3/l10n-central/he/mobile/searchplugins, /builds/slave/m-cen-andrd-l10n-3/mozilla-central/obj-l10n/mobile/locales, /builds/slave/m-cen-andrd-l10n-3/mozilla-central/obj-l10n/mobile/locales
make[3]: *** [searchplugins] Error 1


Attachment #587270 - Flags: review?(aki) → review+, marking FIXED.
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Comment on attachment 587270 [details] [diff] [review]
generate mobile/locales/Makefile

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: no fennec l10n builds for 11
Testing completed (on m-c, etc.): it built, aki's doing the release integration of this and gave the r+
Risk to taking this patch (and alternatives if risky): no risk really, just creating more makefiles early in the build run
Attachment #587270 - Flags: approval-mozilla-aurora?
Comment on attachment 587270 [details] [diff] [review]
generate mobile/locales/Makefile

[Triage Comment]
Approved for Aurora to get fennec l10n builds.
Attachment #587270 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Verified fixed on:

Mozilla/5.0 (Android;Linux armv7l;rv:11.0a2)Gecko/20120130
Firefox/11.0a2 Fennec/11.0a2
Device: Samsung Galaxy S
OS: Android 2.2

Mozilla/5.0 (Android;Linux armv7l;rv:12.0a1)Gecko/20120130
Firefox/12.0a1 Fennec/12.0a1
Device: Samsung Galaxy S
OS: Android 2.2
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.