Closed
Bug 1068382
Opened 10 years ago
Closed 10 years ago
Switch to use multilocale.js for every build
Categories
(Firefox OS Graveyard :: Gaia::L10n, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: zbraniecki, Assigned: zbraniecki)
References
Details
Attachments
(1 file)
We currently only run multilocale if LOCALE_BASEDIR is set.
This means that for builds without that, we kind of don't have a good place in our build system to resolve the magic of branding and device_type and we don't localize manifest.webapp.
I'd like to switch the system to use multilocale processing wherever we use LOCALE_BASEDIR or not.
Assignee | ||
Comment 1•10 years ago
|
||
It actually will be useful for bug 1115807 so that we can handle meta localization in multilocale. Taking.
Assignee | ||
Comment 2•10 years ago
|
||
Very simple change. I basically enable multilocale, but don't do anything in it unless LOCALE_BASEDIR is specified. With those changes running getResources will work without basedir, but I don't need it for now.
The biggest change is that with multilocale running for non basedir enabled build, I can then add meta localization in bug 1115807 and in the future we will be able to easily turn l10n resource gathering for non-basedir builds here.
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8541826 [details]
pull request
Alexandre, can you review this? It basically turns on multilocale.js for builds without LOCALE_BASEDIR in order to centralize l10n related functionality for all builds in that script.
It shouldn't cost any more time because still the l10nManager.localize only runs the actual code for builds with the basedir, but in bug 1115807 I will add one more piece of logic to localize <meta> key in HTML files and this one will be enabled for all builds.
Attachment #8541826 -
Flags: review?(poirot.alex)
Comment 4•10 years ago
|
||
Comment on attachment 8541826 [details]
pull request
Looks good.
Attachment #8541826 -
Flags: review?(poirot.alex) → review+
Comment 5•10 years ago
|
||
This should be also very useful for bug 1101632 and for running the pseudolocales logic on buildtime. Thanks, Zibi!
Blocks: 1101632
Assignee | ||
Comment 6•10 years ago
|
||
Commit: https://github.com/mozilla-b2g/gaia/commit/5bef98fc59306aa8cbbf7285448505c8dab03a21
Merge: https://github.com/mozilla-b2g/gaia/commit/f56224529f88933bcffef89981dfe2d90fa98038
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•