537481 - Don't ship dictionaries with language packs

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect)

Description

[Robert Kaiser]

SeaMonkey currently ship dictionaries with language packs but unfortunately, the licensing story there is somewhat unclear and it's probably not optimal to lose a selected dictionary when switching locales, so it's best to follow what Firefox and Thunderbird are doing and just exclude dictionaries from SeaMonkey builds.

Comment 1

[Robert Kaiser]

Attachment: don't include dictionaries in langpacks

This should be it. I don't have a locale around that builds and/or has a dictionary right now, so I can't test it atm, but normal build is not broken and this should be trivial.

Axel, there's nothing else where we should refer to this, right?

Comment 2

[Axel Hecht [:Pike]]

Comment on attachment 419767 [details] [diff] [review]
don't include dictionaries in langpacks

The clobber-zip piece should stay, otherwise you're breaking the dictionaries for the binary builds.

I guess the trailing ';' can go, too.

Compare with the target in l10n.mk, http://mxr.mozilla.org/mozilla-central/source/toolkit/locales/l10n.mk#175

The only difference should be the searchplugins there.

Comment 3

[Robert Kaiser]

Attachment: v2 - fully sync with what Firefox does

Thanks for looking at this, Axel. I actually think our searchplugins don't even work from an extension, so it's probably better to not package them as well and only have stuff in there that actually works (i.e. chrome only).

With that, the new patch should fully sync our langpacks to those Firefox ships (and make conversion to l10n.mk easier in the future).

Comment 4

[Axel Hecht [:Pike]]

Comment on attachment 419781 [details] [diff] [review]
v2 - fully sync with what Firefox does

Sounds good.

Comment 5

[Robert Kaiser]

Pushed on trunk as http://hg.mozilla.org/comm-central/rev/d8bdb40b098a

Comment 6

[Robert Kaiser]

Pushed on branch as http://hg.mozilla.org/releases/comm-1.9.1/rev/eeb567bb0464