build.sh changes to accomodate gecko multilocale

RESOLVED FIXED in B2G C2 (20nov-10dec)

Status

Firefox OS
GonkIntegration
P1
normal
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: aki, Assigned: mwu)

Tracking

unspecified
B2G C2 (20nov-10dec)
Dependency tree / graph

Firefox Tracking Flags

(blocking-basecamp:+)

Details

(Reporter)

Description

5 years ago
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

5 years ago
No longer depends on: 812831, 812833, 812835, 812836

Updated

5 years ago
No longer depends on: 817197

Updated

5 years ago
blocking-basecamp: ? → +
(Reporter)

Comment 1

5 years ago
I think we will need bug 817197 to package appropriately.
Ben, is this something you can do?  If not, any suggestions as to who should own this?
Assignee: nobody → bhearsum
I know absolutely nothing about the B2G build system, sorry :(. Maybe Fabrice or one of the other original authors of it?
Assignee: bhearsum → nobody
Michael, can you help there?

Updated

5 years ago
Blocks: 818907
This needs an owner who can actually resolve... asap.  Reaching out to Bob Moss to make sure it happens.
(Assignee)

Comment 6

5 years ago
I got this.
Assignee: nobody → mwu
Priority: -- → P1
(Assignee)

Comment 7

5 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

5 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

5 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
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 10

5 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

5 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

5 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

5 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.
(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.
(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.
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

5 years ago
A clobber build is necessary to pick up L10NBASEDIR.
(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.