Closed Bug 716842 Opened 13 years ago Closed 13 years ago

l10n repacks need the makefile in mobile/locales

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set
normal

Tracking

(firefox11 fixed)

VERIFIED FIXED
Firefox 12
Tracking Status
firefox11 --- fixed

People

(Reporter: Pike, Assigned: Pike)

References

Details

Attachments

(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/Makefile.in. Trivial patch, also pushed to try at https://tbpl.mozilla.org/?tree=Try&rev=a69a530db418 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/libmozglue.so': 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/nightly-mobile-repacks.py", line 191, in <module> options.platform, stage_platform, mobileDirName, en_us_binary_url) File "/builds/slave/m-cen-andrd-l10n-1/scripts/scripts/l10n/nightly-mobile-repacks.py", line 51, in createRepacks buildInfo = downloadNightlyBuild(localeSrcDir, env) File "/builds/slave/m-cen-andrd-l10n-1/scripts/scripts/l10n/../../lib/python/build/download.py", 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/commands.py", line 42, in run_cmd return subprocess.check_call(cmd, **kwargs) File "/tools/python-2.5.1/lib/python2.5/subprocess.py", line 461, in check_call raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command '['make', 'unpack']' returned non-zero exit status 2 I currently suspect http://hg.mozilla.org/mozilla-central/rev/c899dde43144 .
[11:11] <glandium> aki: it suggests you're using a packager.mk that is newer than the build you're using it on Building a new nightly to test on.
Yeah, http://hg.mozilla.org/mozilla-central/diff/cf890c9c3e4c/toolkit/mozapps/installer/packager.mk 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 ) Shipit!
Attachment #587270 - Flags: review?(aki) → review+
Status: NEW → RESOLVED
Closed: 13 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
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: