Closed Bug 459221 Opened 16 years ago Closed 7 years ago

compare-locales: possibility for checking optionally localized modules

Categories

(Mozilla Localizations :: Infrastructure, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: kairo, Unassigned)

References

Details

Attachments

(3 files)

As mentioned in bug 458623 comment #5, once compare-locales is listening to [include_*] sections in l10n.ini as requested in bug 459220, we could even have such sections that list an included module as optional and point to an all-locales file that decides if it should be included for the currently compared locale. This would be especially helpful for SeaMonkey, where L10n of chatzilla and venkman is made optional in the build process exactly in that way.
as the problem described here was triggered many times during the SeaMonkey 2.0 RC1 release process, I thought it is time to solve it somehow.

My proposal means adding an "optional_includes" section to suite/locales/l10n.ini with paths to the extensions that don't necessarily require localization.
Each extensions would have its own l10n.ini file with an "all" entry with the path to "all-locales" in the "general" section.

That way we would have what we wanted to - without breaking old compare-locales versions.

I've already added support for this in my own "compare-locales2":
http://hg.mozilla.org/users/akalla_aviary.pl/silme-patched/rev/68da8ad0b901

I'm not sure if that additional section is the best way to solve this - so I'll be happy to hear better proposals.


Regarding implementation of that in Pike's compare-locales: changes to read and parse that section in the L10nConfigParser would be very similar to changes I made in the L10nConfigParser of compare-locales2. 
What I'm not sure of is where we could filter the unwanted packages...
I'm marking this as WONTFIX after 8 years, feel free to move/reopen as necessary.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: