The default bug view has changed. See this FAQ.

Dictionaries should form part of the locale pack in the installer

RESOLVED FIXED

Status

SeaMonkey
Installer
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

11 years ago
In the SeaMonkey installer we currently have the dictionaries and spellcheck components as one optional spellcheck installation selection. Robert commented on IRC that we should install the dictionaries with the locale xpi, and spellcheck as part of the browser xpi by default, as per FF & TB. The currently installed dictionary is approx 680kb in size so there isn't really much of a space saving by not installing it.
 
I think this is reasonable now that the spellcheck is used a lot more within the suite.
(Assignee)

Comment 1

11 years ago
Created attachment 236399 [details] [diff] [review]
Unix installer patch.
(Assignee)

Comment 2

11 years ago
Created attachment 236401 [details] [diff] [review]
Unix installer patch (checked in)

Missed a file.
(Assignee)

Comment 3

11 years ago
Created attachment 236403 [details] [diff] [review]
Windows installer patch (checked in)
(Assignee)

Updated

11 years ago
Attachment #236399 - Attachment is obsolete: true
(Assignee)

Comment 4

11 years ago
Created attachment 236404 [details] [diff] [review]
os2 installer patch (checked in)
(Assignee)

Updated

11 years ago
Attachment #236401 - Flags: superreview?(neil)
Attachment #236401 - Flags: review?(ajschult)

Updated

11 years ago
Attachment #236401 - Flags: superreview?(neil) → superreview+

Updated

11 years ago
Attachment #236401 - Flags: review?(ajschult) → review+
(Assignee)

Comment 5

11 years ago
I'm planning on checking this in on trunk today. I think it may start the nagios system complaining, so cc'ing some relevant people. The lines highlighted in the file from the link below will need removing if there are warnings given. As it isn't part of the main build system I didn't know if I should be providing a patch or not.

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/tools/tinderbox-configs/monitoring/Seamonkey_trunk.txt&rev=1.2&mark=5,17
(Assignee)

Updated

11 years ago
Attachment #236401 - Attachment description: Unix installer patch → Unix installer patch (checked in)
(Assignee)

Comment 6

11 years ago
Comment on attachment 236404 [details] [diff] [review]
os2 installer patch (checked in)

I've not tested this as I don't have os2, but this is done in a similar way to the unix and windows patches on this bug.
Attachment #236404 - Flags: superreview?(neil)
Attachment #236404 - Flags: review?(mozilla)

Updated

11 years ago
Attachment #236404 - Flags: superreview?(neil) → superreview+

Comment 7

11 years ago
The Windows patch seems to work fine, the spell check program files ended up in browser.xpi, the dictionaries in langenus.xpi.
BTW: The packages-win part of the patch had some fuzz:
Hunk #1 succeeded at 38 (offset 1 line).
Hunk #2 succeeded at 189 (offset 14 lines).
Hunk #3 succeeded at 415 with fuzz 1 (offset 36 lines).
It applied cleanly anyway.
(Assignee)

Comment 8

11 years ago
Comment on attachment 236403 [details] [diff] [review]
Windows installer patch (checked in)

Thanks to Frank for the testing, requesting reviews.
Attachment #236403 - Flags: superreview?(neil)
Attachment #236403 - Flags: review?(neil)

Updated

11 years ago
Attachment #236403 - Flags: superreview?(neil)
Attachment #236403 - Flags: superreview+
Attachment #236403 - Flags: review?(neil)
Attachment #236403 - Flags: review+
(Assignee)

Updated

11 years ago
Attachment #236403 - Attachment description: Windows installer patch → Windows installer patch (checked in)
(Assignee)

Comment 9

11 years ago
ping Michael: any chance of a review on the os2 installer patch sometime?
(Assignee)

Comment 10

11 years ago
Comment on attachment 236404 [details] [diff] [review]
os2 installer patch (checked in)

I'm guessing Michael's away or busy, therefore switching review to Andrew.
Attachment #236404 - Flags: review?(mozilla) → review?(ajschult)

Updated

11 years ago
Attachment #236404 - Flags: review?(ajschult) → review+
(Assignee)

Comment 11

11 years ago
Comment on attachment 236404 [details] [diff] [review]
os2 installer patch (checked in)

os2 patch checked in.
Attachment #236404 - Attachment description: os2 installer patch → os2 installer patch (checked in)
(Assignee)

Updated

11 years ago
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.