Last Comment Bug 358823 - dom_svg.xpt is missing from packages-os2
: dom_svg.xpt is missing from packages-os2
: fixed-seamonkey1.1
Product: SeaMonkey
Classification: Client Software
Component: Installer (show other bugs)
: Trunk
: x86 OS/2
-- normal (vote)
: ---
Assigned To: Peter Weilbacher
Depends on:
  Show dependency treegraph
Reported: 2006-10-30 15:04 PST by Peter Weilbacher
Modified: 2007-01-05 02:49 PST (History)
1 user (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

little cleanup (1.44 KB, patch)
2006-10-30 15:11 PST, Peter Weilbacher
mozilla: review+
Details | Diff | Splinter Review

Description User image Peter Weilbacher 2006-10-30 15:04:13 PST
dom_svg.xpt does not get packaged into OS/2 installer builds as it is not listed in packages-os2. This has the effect that some SVG functionality is missing (which works in a ZIP build).
Comment 1 User image Peter Weilbacher 2006-10-30 15:11:27 PST
Created attachment 244138 [details] [diff] [review]
little cleanup

Adding the missing line is easy enough. chrome.xpt was listed twice, I removed one instance. Should eventually go onto the 1.8 branch, too, where I would also remove the duplicate line imgicon.xpt and the line about the non-existing libimgicon.dll (both were already remove on trunk).

I'm also wondering why bin/components/mozcomps.xpt and bin/components/msgmdn.xpt are commented out in packages-os2 while they are packaged on Unix and Windows.
Comment 2 User image Mike Kaply [:mkaply] 2007-01-04 07:32:54 PST
Comment on attachment 244138 [details] [diff] [review]
little cleanup

No idea on the other files - are those two files definitely in an OS/2 build?
Comment 3 User image Peter Weilbacher 2007-01-05 02:35:04 PST
OK, checked into trunk. You are right, the two other XPTs are not built on OS/2, although I didn't find out why not or what they are for, I added comments for that.
Comment 4 User image Peter Weilbacher 2007-01-05 02:49:30 PST
Checked into 1.8 branch, too, so this fix will be in SeaMonkey 1.1.

Note You need to log in before you can comment on or make changes to this bug.