769490 - Merge metrofx l10n.ini information into browser/locales

Status: RESOLVED FIXED

(Firefox for Metro Graveyard :: General, defect)

Description

[Jim Mathies [:jimm]]

From Axel via an email thread:

> So, as we're having one firefox deliverable that we localize, we'll need 
> to hook up the metro l10n files as part of browser, I think, and not as 
> part of an independent product.
> 
> Thus, there shouldn't be a separate l10n.ini, but the one in 
> browser/locales should reference browser/metro along the other dirs that 
> belong to Firefox itself.
> 
> Asa, I think we'd like to keep langpacks as an option.
> 
> We may need to modify the generic install.rdf in browser to be compat 
> with the metro app, if that's using a different app id.

metro specific locale info is currently in 

http://mxr.mozilla.org/projects-central/source/elm/browser/metro/locales/

Comment 1

[Jim Mathies [:jimm]]

Attachment: patch v.1

Comment 2

[Jim Mathies [:jimm]]

Comment on attachment 694041 [details] [diff] [review]
patch v.1

Axel, I wasn't sure how detailed I needed to be in listing directories under the [compare] section. Will this suffice or do we need a more detailed list that points into the locales directories?

Comment 3

[Axel Hecht [:Pike]]

Comment on attachment 694041 [details] [diff] [review]
patch v.1

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

This looks good, you'll also need to adapt browser/locales/filter.py to take the new directory into account, at http://mxr.mozilla.org/mozilla-central/source/browser/locales/filter.py#8.

Comment 4

[Jim Mathies [:jimm]]

Axel, is there a wiki page describing the steps to do a local repack? I'd like to test the metro bits w/l10n build steps, but haven't found much on what we do here.

Comment 5

[Jim Mathies [:jimm]]

Hmm, I really don't understand what filter.py is supposed to do, so I'm not sure how to modify it.

Comment 6

[Axel Hecht [:Pike]]

https://developer.mozilla.org/en-US/docs/Creating_a_Language_Pack has info on how to do repacks for desktop. I suggest french or italian as example languages.

Also, filter.py controls which files and entries within matter for l10n, as it's not always 1-to-1. You'll need to add 'browser/metro' next to 'browser' in the initial test, and after that things should not need changes. If they would, that'd probably be a bug, like region.properties and search ;-)

Comment 7

[Jim Mathies [:jimm]]

(In reply to Axel Hecht [:Pike] from comment #6)
> https://developer.mozilla.org/en-US/docs/Creating_a_Language_Pack has info
> on how to do repacks for desktop. I suggest french or italian as example
> languages.
> 
> Also, filter.py controls which files and entries within matter for l10n, as
> it's not always 1-to-1. You'll need to add 'browser/metro' next to 'browser'
> in the initial test, and after that things should not need changes. If they
> would, that'd probably be a bug, like region.properties and search ;-)

Thank you, will update.

I've been poking through logs and pushing stuff at try. Found pymake l10n-check which has helped me step through the pack/unpack process.

Comment 8

[Jim Mathies [:jimm]]

Attachment: patch v.2

Comment 9

[Axel Hecht [:Pike]]

Comment on attachment 694876 [details] [diff] [review]
patch v.2

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

Looks good, thanks.

Comment 10

[Jim Mathies [:jimm]]

https://hg.mozilla.org/projects/elm/rev/b5c0a22549e6

Comment 11

[Matt Brubeck (:mbrubeck)]

Resolving bugs in the Firefox for Metro product that are fixed on the elm branch.  Sorry for the bugspam.  Search your email for "bugspam-elm" if you want to find and delete all of these messages at once.

Comment 12

[Jim Mathies [:jimm]]

https://hg.mozilla.org/mozilla-central/rev/58f7533b8288