Status

()

Firefox for Android
General
RESOLVED WONTFIX
6 years ago
6 years ago

People

(Reporter: elan, Assigned: dougt)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [birch] [Product Approved])

Attachments

(2 attachments, 2 obsolete attachments)

(Reporter)

Description

6 years ago
* nightly builds are already being generated; need to generate snippets?
* only needed for nightly/aurora
* not required, but useful for testing/dogfooding
* Need to turn on check for updates
(Reporter)

Updated

6 years ago
Assignee: nobody → doug.turner
(Reporter)

Updated

6 years ago
Whiteboard: [birch] → [birch] [Product Approved]
(Assignee)

Comment 1

6 years ago
See https://bugzilla.mozilla.org/show_bug.cgi?id=694077#c20 for what has to be done on the client side.
(Assignee)

Comment 2

6 years ago
Created attachment 568845 [details] [diff] [review]
short term work around
Attachment #568845 - Flags: review?(fabrice)
(Assignee)

Comment 3

6 years ago
Created attachment 568846 [details] [diff] [review]
patch v. 1

This works for android, but not sure about other platforms.  needs to go to try.  Ted, does this look about right?
Attachment #568846 - Flags: review?(ted.mielczarek)
Comment on attachment 568845 [details] [diff] [review]
short term work around

Review of attachment 568845 [details] [diff] [review]:
-----------------------------------------------------------------

r+ since the real fix is in the second patch.
Attachment #568845 - Flags: review?(fabrice) → review+
(Assignee)

Updated

6 years ago
Attachment #568846 - Flags: review?(ted.mielczarek) → review?(robert.bugzilla)
Comment on attachment 568846 [details] [diff] [review]
patch v. 1

Doug, I'm slammed for at least this week so if you could get khuey or Ted to take a look at this it will likely happen faster.

Couple of quick comments.
I think it would be a good thing to be available even with builds that don't include the updater.
I think it would be best to name it platformLocale or something similar instead of locale to avoid ambiguity with the locale currently in use.
Blocks: 686480
FTR, the attached patch applies on birch, not on m-c. It is on top of the hack that landed on birch for bug 686466.
Depends on: 686466
Created attachment 569996 [details] [diff] [review]
Embed locale information in nsXREAppData instead of shipping an update.locale file. Original patch by dougt

Pushed to try:
https://tbpl.mozilla.org/?noignore=1&tree=Try&rev=1a3aff53cc8c

I didn't include the part adding locale information from platform.ini because the updater actually doesn't look there. The updater doesn't seem to be able to care about application and XRE separately anyways. So only one information is required, and that is that of the locale bundled in the application. Thus the name of the variable.
Attachment #569996 - Flags: review?(khuey)
Attachment #568846 - Attachment is obsolete: true
Attachment #568846 - Flags: review?(robert.bugzilla)
Comment on attachment 569996 [details] [diff] [review]
Embed locale information in nsXREAppData instead of shipping an update.locale file. Original patch by dougt

It breaks all tests on windows and xpcshell on all platforms
Attachment #569996 - Flags: review?(khuey)
(Assignee)

Updated

6 years ago
Priority: P1 → --
Summary: App Updates → update.locale support
Created attachment 570193 [details] [diff] [review]
Embed locale information in nsXREAppData instead of shipping an update.locale file. Original patch by dougt

Was missing an update to the script generating application.ini.h

https://tbpl.mozilla.org/?noignore=1&tree=Try&rev=0a8e1e439f59
Attachment #569996 - Attachment is obsolete: true
Still breaks windows :(
Now that I think of it, hardcoding the locale in the binary is not going to work with l10n repacking :-/

So in the end, bug 696555 is probably the way to go.
<rs> glandium: then moving the file into omni.jar or a similar method sounds like the only course available

We'll thus go the bug 696555 way.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.