Closed Bug 344236 Opened 16 years ago Closed 16 years ago

Remove xpinstall-based installer code

Categories

(Toolkit Graveyard :: Build Config, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9alpha1

People

(Reporter: benjamin, Assigned: benjamin)

References

Details

Attachments

(1 file)

The xpinstall-based toolkit installer can almost go. The Linux code is currently unused, and the Windows code is only used by Thunderbird and (I think?) will be ported to use the NSIS installer soon.
Thunderbird moved over, AFAICT, does this affect SeaMonkey?

I filed bug 345846, which I would hope to land on the 1.8 branch, as it makes localization easier quite a bit, in the light of new locales on 2.0.0.x, too. So the question I'd have is, does it make sense to have that bug open separately or is it a plain dupe?
Keep that open separately... I have no plans to do this on the 1.8branch, and it's relatively low priority.

This doens't affect seamonkey at the moment because they are still using the xpinstall/... installer.
This is almost everything. I didn't know who used some of the scripts in toolkit/mozapps/installer, so I didn't touch those yet.

diff -u4 2>&1 so that you can see what I removed without having to wade through the actual contents of the files.
Attachment #234776 - Flags: first-review?(robert.bugzilla)
Benjamin, I'd like to hold off on this until bug 349551 is fixed.
Depends on: 349551
Comment on attachment 234776 [details] [diff] [review]
Remove xpinstall-based installer, rev. 1

>Index: toolkit/mozapps/Makefile.in
...
>-# XXX SUITE doesn't want to build the toolkit installer yet
>-ifndef MOZ_SUITE
>-# The installer is built during the packaging phase, but the uninstaller needs
>-# to be built earlier.
>-ifdef MOZ_INSTALLER
>-ifeq ($(OS_ARCH),WINNT)
>-DIRS += installer/windows/wizard/uninstall
>-endif
>-endif
>-endif
removed by the patch in bug 349551

>cvs diff: toolkit/mozapps/installer/windows/wizard/setup/setup.ico was removed, no comparison available
setup.ico is still needed and is moved to toolkit/mozapps/installer/windows/nsis by the patch in bug 349551

toolkit/mozapps/installer/windows/install_sub.pl can also be removed
toolkit/mozapps/installer/Makefile.in can also be removed. 

allmakefiles.sh needs to be updated for these changes
-toolkit/mozapps/installer/unix/wizard/Makefile
-toolkit/mozapps/installer/unix/Makefile
-toolkit/mozapps/installer/Makefile
-toolkit/mozapps/installer/windows/Makefile
-toolkit/mozapps/installer/windows/wizard/Makefile
-toolkit/mozapps/installer/windows/wizard/setup/Makefile
-toolkit/mozapps/installer/windows/wizard/setuprsc/Makefile
-toolkit/mozapps/installer/windows/wizard/uninstall/Makefile

It would be nice to also remove the following from allmakefiles.sh at the same time
extensions/inspector/build/src/Makefile
mozilla/browser/extensions/package-fixup/Makefile.

If you like I can probably do the browser portion in about a week.

r=me but please wait on bug 349551 landing before landing this.
Attachment #234776 - Flags: first-review?(robert.bugzilla) → first-review+
Fixed on trunk.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Almost got them all... I missed the following icon that can be removed
mozilla/toolkit/mozapps/installer/windows/wizard/setuprsc/setup.ico
Removed /toolkit/mozapps/installer/windows/wizard/setuprsc/setup.ico with a verbal r= from bsmedberg on irc
Blocks: 345846
Product: Toolkit → Toolkit Graveyard
You need to log in before you can comment on or make changes to this bug.