Closed
Bug 818560
Opened 12 years ago
Closed 12 years ago
build.sh changes to accomodate gecko multilocale
Categories
(Firefox OS Graveyard :: GonkIntegration, defect, P1)
Firefox OS Graveyard
GonkIntegration
Tracking
(blocking-basecamp:+)
People
(Reporter: mozilla, Assigned: mwu)
References
Details
We don't seem to have a way to run individual make commands mid-build, so afaict we need to add multilocale build+package steps to build.sh.
This will most likely be conditional on environment variables, so the default developer build stays as-is.
This blocks multiple blocking-basecamp+ C2 bugs, so marking as such.
Reporter | ||
Updated•12 years ago
|
Updated•12 years ago
|
blocking-basecamp: ? → +
Reporter | ||
Comment 1•12 years ago
|
||
I think we will need bug 817197 to package appropriately.
Comment 2•12 years ago
|
||
Ben, is this something you can do? If not, any suggestions as to who should own this?
Assignee: nobody → bhearsum
Comment 3•12 years ago
|
||
I know absolutely nothing about the B2G build system, sorry :(. Maybe Fabrice or one of the other original authors of it?
Assignee: bhearsum → nobody
Comment 4•12 years ago
|
||
Michael, can you help there?
Comment 5•12 years ago
|
||
This needs an owner who can actually resolve... asap. Reaching out to Bob Moss to make sure it happens.
Updated•12 years ago
|
Priority: -- → P1
Assignee | ||
Comment 7•12 years ago
|
||
I'll try to get this all fixed up on Friday (7th). This mostly depends on bug 817197 to do most of the work and then I'll add some bits to gonk-misc to make it all stick together. We'll need to figure out how to make the l10n stuff work for our friends though - maybe add git mirrors for the l10n stuff and get this into the manifest.
Comment 8•12 years ago
|
||
For developers, we're keeping the few locales in gaia's git, and they should be fine, even if the network errors are not translated.
For downstream parties, they'll work off of hg, as does our automation infrastructure.
As for git mirrors for l10n, there are many reasons against doing those, don't want to clutter this bug with that discussion. I don't see a need for them here, though, too.
Assignee | ||
Comment 9•12 years ago
|
||
The appropriate pieces have landed in gonk-misc and B2G.
All you have to do is set MOZ_CHROME_MULTILOCALE and L10NBASEDIR in the environment and the B2G build system will take care of the rest. These two can be set in the .userconfig. For example I have this in my userconfig:
L10NBASEDIR=$PWD/l10n-base
MOZ_CHROME_MULTILOCALE="ja zh-TW"
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 10•12 years ago
|
||
That'd be https://github.com/mozilla-b2g/gonk-misc/commit/e63bbf6c3f5a116b7b605e4d2bd93c84bd4b99ad and https://github.com/mozilla-b2g/B2G/commit/41c17a6abfd5f488ec99d9aa246f5b07583403c7, right?
Does that trigger what we need to trigger for the gaia side?
Comment 11•12 years ago
|
||
The patches I found are not related to gecko repos, so unblocking the uplift bug 818907.
No longer blocks: 818907
Assignee | ||
Comment 12•12 years ago
|
||
(In reply to Axel Hecht [:Pike] from comment #10)
> Does that trigger what we need to trigger for the gaia side?
I haven't touched the gaia side at all. As far as I can tell, gaia already installs its set of locales without any intervention. I was enjoying the gaia lockscreen in zh-TW.
Assignee | ||
Comment 13•12 years ago
|
||
(In reply to Axel Hecht [:Pike] from comment #11)
> The patches I found are not related to gecko repos, so unblocking the uplift
> bug 818907.
Note that releng will need to get their gonk toolchain/codedrop updated to start supporting this.
Comment 14•12 years ago
|
||
(In reply to Michael Wu [:mwu] from comment #12)
> (In reply to Axel Hecht [:Pike] from comment #10)
> > Does that trigger what we need to trigger for the gaia side?
>
> I haven't touched the gaia side at all. As far as I can tell, gaia already
> installs its set of locales without any intervention. I was enjoying the
> gaia lockscreen in zh-TW.
Yeah - the Gaia build system looks at LOCALE_BASEDIR and LOCALES_FILE. We enabled that in the RelEng builds a couple of days ago. Probably need to rename some of these vars at some point to avoid confusion, but it sounds like we're OK for now since they don't overlap.
Comment 15•12 years ago
|
||
(In reply to Michael Wu [:mwu] from comment #13)
> (In reply to Axel Hecht [:Pike] from comment #11)
> > The patches I found are not related to gecko repos, so unblocking the uplift
> > bug 818907.
>
> Note that releng will need to get their gonk toolchain/codedrop updated to
> start supporting this.
Filed bug 819979 for this.
Comment 16•12 years ago
|
||
I'm having a bit of trouble with this. I have L10NBASEDIR and MOZ_CHROME_MULTILOCALE set in the environment, but I get the following error when building:
07:42:05 INFO - make -C gecko -f client.mk -s
07:42:06 INFO - Adding client.mk options from /builds/slave/b2g-m-cen-otoro-dep/build/gonk-misc/default-gecko-config:
07:42:06 INFO - MOZ_OBJDIR=/builds/slave/b2g-m-cen-otoro-dep/build/objdir-gecko
07:42:06 INFO - MOZ_MAKE_FLAGS=-s -j16
07:42:06 INFO - TEST-PASS | check-sync-dirs.py | /builds/slave/b2g-m-cen-otoro-dep/build/gecko/js/src/config <= /builds/slave/b2g-m-cen-otoro-dep/build/gecko/config
07:42:06 INFO - make[3]: warning: -jN forced in submake: disabling jobserver mode.
07:42:06 ERROR - /builds/slave/b2g-m-cen-otoro-dep/build/gecko/config/config.mk:693: *** L10NBASEDIR not defined by configure. Stop.
07:42:06 INFO - make[3]: INTERNAL: Exiting with 17 jobserver tokens available; should be 16!
07:42:06 ERROR - make[2]: *** [realbuild] Error 2
07:42:06 ERROR - make[1]: *** [build] Error 2
I'm trying to debug, but any pointers or suggestions would be appreciated.
Assignee | ||
Comment 17•12 years ago
|
||
A clobber build is necessary to pick up L10NBASEDIR.
Comment 18•12 years ago
|
||
(In reply to Michael Wu [:mwu] from comment #17)
> A clobber build is necessary to pick up L10NBASEDIR.
That got me past the error - thanks.
You need to log in
before you can comment on or make changes to this bug.
Description
•