349159 - Add url formatter items to SeaMonkey installers.

Status: RESOLVED FIXED

(SeaMonkey :: Installer, defect)

Description

[Mark Banner (:standard8)]

Bug 347944 introduced a url formatter to toolkit, but only added the items to ff's installer. As it is in toolkit and built for everyone, we need to include it in the SeaMonkey installers for when we are a toolkit app.

Comment 1

[Mark Banner (:standard8)]

Attachment: Add missing options for unix packages

This adds the missing unix item definitions, os2 definitions are contained within bug 332203, windows definitions will be included in the patch that's being done for the windows installer for suiterunner.

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 234454 [details] [diff] [review]
Add missing options for unix packages

Bug 347944 got the xpt name wrong :-( new patch coming up.

Comment 3

[Mark Banner (:standard8)]

Attachment: Fix unix and os2 packaging lists (checked in).

Corrected the unix patch for the xpt name and added in the change for the os2 packager (as I'd already done the os2 on bug 332203).

Comment 4

[Mark Banner (:standard8)]

btw I'm leaving this open until we've fixed up the windows installer (probably to use nsis) just in case we forget to add these items (note xpfe doesn't need these).

Comment 5

[Mark Banner (:standard8)]

Attachment: Add to windows packaging lists (checked in)

suiterunner is currently planning on using the NSIS installer - However, just in case we decide to required urlformatter on xpfe builds (as bug 351152 may do), and so we can close out this bug, I'd like to add it to the windows packaging lists anyway as it won't actually harm anything to have it in the lists but not present.

Comment 6

[[e-mail]]

Comment on attachment 240275 [details] [diff] [review]
Add to windows packaging lists (checked in)

Doesn't this need to be done for other platforms too? (check with ajschult?)

Comment 7

[Mark Banner (:standard8)]

(In reply to comment #6)
> (From update of attachment 240275 [details] [diff] [review] [edit])
> Doesn't this need to be done for other platforms too? (check with ajschult?)

See the first patch on this bug - that has already done unix & os2 as they will still be used for suiterunner.

Comment 8

[Mark Banner (:standard8)]

Comment on attachment 240275 [details] [diff] [review]
Add to windows packaging lists (checked in)

/cvsroot/mozilla/xpinstall/packager/packages-win,v  <--  packages-win
new revision: 1.309; previous revision: 1.308
done