Clean up SeaMonkey packages file

RESOLVED FIXED in seamonkey2.0a1

Status

SeaMonkey
Installer
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: mcsmurf, Assigned: mcsmurf)

Tracking

Trunk
seamonkey2.0a1
x86
Windows XP
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Assignee)

Description

11 years ago
The SeaMonkey packages file still includes references to some files which are no longer built (see Windows SeaMonkey tinderbox log), this should be changed. I'll add further comments and Bug #s to the bug here as time moves on.
Current changes to suite/installer/windows/packages:
Remove extensions\inspector@mozilla.org\components\inspector.dll and extensions\inspector@mozilla.org\components\inspector.xpt (Bug 338746)
Remove gkgfxwin.dll: this was for the old GFX toolkit
Remove nsprefm.dll: this file was built by profile/pref-migrator/src/Makefile.in, which is only in the build process if MOZ_SINGLE_PROFILE is not set. MOZ_SINGLE_PROFILE is set to 1 when MOZ_XUL_APP is set.
(Assignee)

Comment 1

11 years ago
Remove editor.dll: editor was moved to gklayout as part of Bug 347664
Remove intlcmpt.dll/intlcmpt.xpt: intlcmpt was removed from the build as part of Bug 352180
(Assignee)

Comment 2

11 years ago
Remove mozucth.dll/mozucth.xpt: mozucth was already removed from the build in 2002(!), checkin comment in CVS "bug 156723 - remove xfer and ucth from the build - no longer used r=law, sr=blake, a=asa" that bug does not contain anything about ucth itself though
(Assignee)

Updated

11 years ago
Blocks: 382571
(Assignee)

Comment 3

11 years ago
Created attachment 266764 [details] [diff] [review]
Clean-up, part 1

This patch here should also fix Bug 382571.
(Assignee)

Updated

11 years ago
Attachment #266764 - Flags: review?(bugzilla)
The palm sync code has also moved (at the switch over, its now an extension), so at some stage we also need to do the following:

Remove:

;these Palm Sync files are conditionally built so would not exist unless built
bin\mozABConduit.dll
bin\PalmSyncProxy.dll
bin\PalmSyncInstall.exe
bin\components\palmsync.dll
bin\components\palmsync.xpt
bin\palm.html

Add:

bin\extensions\p@m\install.rdf
bin\extensions\p@m\mozABConduit.dll
bin\extensions\p@m\PalmSyncInstall.exe
bin\extensions\p@m\PalmSyncProxy.dll
bin\extensions\p@m\palmsync.dll
bin\extensions\p@m\palmsync.xpt
bin\extensions\p@m\defaults\preferences\palmsync.js
Comment on attachment 266764 [details] [diff] [review]
Clean-up, part 1

r=me, (I'm happy for this to be checked in) but please investigate whether or not we need to make the following changes as well - I don't know if they do, I just picked up they were in the res directory or packages file and not in the res directory:

remove bin\res\rdf\{...}
add bin\res\cmessage.txt
add bin\res\bloatcycle.html
add bin\res\html\*


Is this likely to fix bug 382570 as well? If so please add dep.
Attachment #266764 - Flags: review?(bugzilla) → review+

Comment 6

11 years ago
(In reply to comment #5)
> (From update of attachment 266764 [details] [diff] [review])
[...]
> Is this likely to fix bug 382570 as well? If so please add dep.
> 
No, the first Patch will not fix Bug 382570 as well. I have just tried and copied the files:

bin\res\langGroups.properties
bin\res\language.properties

from an older tpol-Build into the ..\res Directory from Build 2007060102, and this have fixed Bug 382571 for me, but Bug 382570 was still alive. 

Also I am a little bit confused, why in tpol-Suiterunner-Builds (2007052023) there are 243 Objects in the ..\components directory, and in new Suiterunner-Trunk-Builds from sea-win32-tbox Clbr VM-release-Box only 95 Objects?


(Assignee)

Comment 7

11 years ago
Created attachment 266932 [details] [diff] [review]
Clean-up, part 1 (as checked in)

Includes the changes from the review comment. bloatcycle.html is for tinderbox tests only. cmessage.txt seems to be used, Thunderbird does not package it though (dunno for what reason, FF does).
(Assignee)

Comment 8

11 years ago
Remove AcctIdl.dll/AcctIdl.xpt: Was already removed from the source tree 2001, see Bug 103054
Remove nativapp.dll: Removed as part of Bug 96524
Fix typo imcicon.xpt<->imgicon.xpt
(Assignee)

Comment 9

11 years ago
Remove pipnss.jar: Only exists when building a non-MOZ_XUL_APP application, see http://lxr.mozilla.org/mozilla/source/security/manager/ssl/resources/jar.mn
Remove imgxbm.dll: Merged to imglib2.dll as part of Bug 168048
Remove imgbmp.dll: Merged to imglib2.dll as part of Bug 168048
Depends on: 382438
Depends on: 383402
(Assignee)

Updated

11 years ago
Blocks: 383402
No longer depends on: 383402
(Assignee)

Comment 10

11 years ago
Created attachment 267418 [details] [diff] [review]
Clean-up, part 2
Attachment #267418 - Flags: review?(bugzilla)
Comment on attachment 267418 [details] [diff] [review]
Clean-up, part 2

This looks ok, r=me. Don't forget bug 382438 as well (fastfind.xpt), but we can do that in another patch.
Attachment #267418 - Flags: review?(bugzilla) → review+
(Assignee)

Comment 12

11 years ago
Comment on attachment 267418 [details] [diff] [review]
Clean-up, part 2

Checked in (with the fastfind.xpt change).
Created attachment 280196 [details] [diff] [review]
More clean up (checked in)

Rather than create a new bug, I'll just hi-jack this one. Here's some more changes I found, mostly removals.

I'm not quite sure why Frank added bin\components\bayesflt.dll - its a dll for non-static mailnews, and as we're building static, we don't need it.
Attachment #280196 - Flags: review?(bugzilla)
(Assignee)

Comment 14

11 years ago
Comment on attachment 280196 [details] [diff] [review]
More clean up (checked in)

Looks ok. Resolve the bug as FIXED when the patch is checked in?
Attachment #280196 - Flags: review?(bugzilla) → review+
Attachment #280196 - Attachment description: More clean up → More clean up (checked in)
(Assignee)

Comment 15

11 years ago
All fixed now.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED

Updated

11 years ago
Target Milestone: --- → seamonkey2.0alpha
You need to log in before you can comment on or make changes to this bug.