Closed Bug 537481 Opened 12 years ago Closed 12 years ago

Don't ship dictionaries with language packs


(SeaMonkey :: Build Config, defect)

Not set


(Not tracked)



(Reporter: kairo, Assigned: kairo)



(Keywords: fixed-seamonkey2.0.3)


(1 file, 1 obsolete file)

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.
Flags: blocking-seamonkey2.0.2?
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?
Attachment #419767 - Flags: review?(l10n)
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,

The only difference should be the searchplugins there.
Attachment #419767 - Flags: review?(l10n) → review-
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 easier in the future).
Attachment #419767 - Attachment is obsolete: true
Attachment #419781 - Flags: review?(l10n)
Comment on attachment 419781 [details] [diff] [review]
v2 - fully sync with what Firefox does

Sounds good.
Attachment #419781 - Flags: review?(l10n) → review+
Attachment #419781 - Flags: approval-seamonkey2.0.2?
Pushed on trunk as
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
Attachment #419781 - Flags: approval-seamonkey2.0.3? → approval-seamonkey2.0.3+
Pushed on branch as
Flags: blocking-seamonkey2.0.3?
You need to log in before you can comment on or make changes to this bug.