Closed
Bug 1460952
Opened 7 years ago
Closed 7 years ago
Clean up dictionaries in l10n repos
Categories
(Mozilla Localizations :: Other, enhancement)
Mozilla Localizations
Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: flod, Assigned: flod)
References
Details
I confess I've never looked at the dictionaries shipping across languages, and there seems to be a lot of space for clean ups.
https://dxr.mozilla.org/l10n-central/search?q=path%3A.dic&redirect=false
Possible broken locales:
> el/extensions/spellcheck/el.dic
Would this even work, out of the hunspell folder? Need to try a Greek build.
Locales with en-US dictionary:
> ja-JP-mac/extensions/spellcheck/hunspell/en-US.dic
> ja/extensions/spellcheck/hunspell/en-US.dic
> km/extensions/spellcheck/hunspell/en-US.dic
> lij/extensions/spellcheck/hunspell/en-US.dic
> or/extensions/spellcheck/hunspell/en-US.dic
> ta/extensions/spellcheck/hunspell/en-US.dic
> zh-TW/extensions/spellcheck/hunspell/en-US.dic
I think these were copied by localizers at the time, when working directly on hg. Probably worth filing a bug for ja and zh-TW, but the others should just be removed (IMO).
zh-TW was copied from ja, and never updated
https://hg.mozilla.org/l10n-central/zh-TW/log/tip/extensions/spellcheck/hunspell/en-US.dic
The Japanese one is years behind
https://hg.mozilla.org/l10n-central/ja/log/tip/extensions/spellcheck/hunspell/en-US.dic
Locales with a lot of extra "stuff" that should just be removed.
https://hg.mozilla.org/l10n-central/km/file/tip/extensions/spellcheck/hunspell/dictionary-sources
https://hg.mozilla.org/l10n-central/lij/file/tip/extensions/spellcheck/hunspell/dictionary-sources
| Assignee | ||
Comment 1•7 years ago
|
||
(In reply to Francesco Lodolo [:flod] from comment #0)
> I think these were copied by localizers at the time, when working directly
> on hg. Probably worth filing a bug for ja and zh-TW, but the others should
> just be removed (IMO).
To clarify: I don't think it was a conscious choice to add it for any of these languages, with the exception of ja and zh-TW.
In these two cases, I'd like to understand why it's useful to ship an English dictionary, in particular considering that nobody is updating it, and there are better choices available on AMO.
| Assignee | ||
Comment 2•7 years ago
|
||
Remove dictionary-sources from lij, km
https://hg.mozilla.org/l10n-central/lij/rev/75f88be20670127782a43edaddef371ff986962b
https://hg.mozilla.org/l10n-central/km/rev/d103ecb26664fa357656f3ceae101410d25070f1
And discovered Georgian (ka), shipping en-US but calling it Georgian…
https://hg.mozilla.org/l10n-central/ka/file/tip/extensions/spellcheck/hunspell/readme
Will file a bug for ja and zh-TW to start.
| Assignee | ||
Comment 3•7 years ago
|
||
I think this is as much we could clean up.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 4•7 years ago
|
||
(In reply to Francesco Lodolo [:flod] from comment #0)
> Locales with en-US dictionary:
>
> > ja-JP-mac/extensions/spellcheck/hunspell/en-US.dic
> > ja/extensions/spellcheck/hunspell/en-US.dic
> > km/extensions/spellcheck/hunspell/en-US.dic
> > lij/extensions/spellcheck/hunspell/en-US.dic
> > or/extensions/spellcheck/hunspell/en-US.dic
> > ta/extensions/spellcheck/hunspell/en-US.dic
> > zh-TW/extensions/spellcheck/hunspell/en-US.dic
>
> I think these were copied by localizers at the time, when working directly
> on hg. Probably worth filing a bug for ja and zh-TW, but the others should
> just be removed (IMO).
I realized that, in the end, I didn't really remove them for km, lij, or, ta.
https://hg.mozilla.org/l10n-central/km/rev/5884668360830cc123c43d2121e49f92164b891f
https://hg.mozilla.org/l10n-central/lij/rev/465e7e161ec5efd837e4d31df49ad3867ebeec7a
https://hg.mozilla.org/l10n-central/or/rev/8c6d61be98470e4807068ef5a0a1d1b9603fe4d3
https://hg.mozilla.org/l10n-central/ta/rev/4ab976c7a777f2157524aab49eb6ee16c6464073
Rationale for removing en-US:
- We're working on improving feature discoverability
- These languages don't have a maintainer on Mercurial, and this dictionary is obsolete
You need to log in
before you can comment on or make changes to this bug.
Description
•