Closed
Bug 386903
Opened 16 years ago
Closed 16 years ago
remove xpinstall/packager from cvs
Categories
(Core Graveyard :: Installer: GRE, defect)
Core Graveyard
Installer: GRE
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kairo, Assigned: kairo)
Details
Attachments
(1 file)
841 bytes,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
SeaMonkey is now using NSIS installer for Windows, and while it would be nice to have a system for making some modules optional for a Linux install, the old xpinstall way is probably not the way to do it. As bsmedberg mentioned on IRC one time, xpinstall/packager should probably just be removed.
Comment 1•16 years ago
|
||
Note that there still are some files in there used during toolkit packaging, notably: xptlink.pl (see http://mxr.mozilla.org/mozilla/source/toolkit/mozapps/installer/packager.mk#311) Packager.pm (see http://mxr.mozilla.org/mozilla/source/toolkit/mozapps/installer/packager.mk#285) But all the rest should go, good riddance and all that.
![]() |
Assignee | |
Comment 2•16 years ago
|
||
So all other hits from http://mxr.mozilla.org/mozilla/search?string=xpinstall%2Fpackager can be safely ignored?
![]() |
Assignee | |
Comment 3•16 years ago
|
||
OK, this patch removes the Makefiles from allmakefiles.sh Additionally, the review should include cvs removing everything from xpinstall/packager, with the exception of the following files: xpinstall/packager/Packager.pm xpinstall/packager/xptlink.pl I'm not completely sure about those files, please include a review comment if they should stay or can be cvs removed: xpinstall/packager/pkgcp.pl (referenced from embedding/config/Makefile.in and extensions/java/xpcom/package/Makefile.in) xpinstall/packager/common/share.t (referenced from toolkit/mozapps/installer/makejs.pl)
Comment 4•16 years ago
|
||
pkgcp.pl should stay... Camino still uses embedding/config, pox on it I don't know what extensions/java/xpcom needs it for, cc'ing Javier share.t can go AFAIK toolkit/mozapps/installer/makejs.pl is not used, and in fact we should remove it along with a bunch of other cruft there: makeall.pl makexpi.pl parseandmakejs.pl
Updated•16 years ago
|
Attachment #271529 -
Flags: review?(benjamin) → review+
![]() |
Assignee | |
Comment 5•16 years ago
|
||
Thanks. This has been checked in, per IRC I also did remove the files mentioned in comment #4
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 6•16 years ago
|
||
Looks like you missed these: http://mxr.mozilla.org/seamonkey/source/xpinstall/packager/common/ http://mxr.mozilla.org/seamonkey/source/xpinstall/packager/os2/ http://mxr.mozilla.org/seamonkey/source/xpinstall/packager/unix/
Comment 7•16 years ago
|
||
OS/2 was still happily using the stuff you removed. But this forces us to think about other installer options sooner rather than later. Is there a similar bug about removing xpinstall/wizard/?
![]() |
Assignee | |
Comment 8•16 years ago
|
||
Peter: I'm somewhat sorry for catching you without specific warnings, but it's been the plan to get rid of XPI-based installers for some time now. xpinstall/wizard is bug 387450 - should I leave the os2 dirs around (as I missed the one here anyways) or are you happy to get anything you need from older states of CVS?
Comment 9•16 years ago
|
||
kairo: please remove them, without packages-os2 they useless now anyway, and I think that's the (only?) file we would have to start from for a new installer.
![]() |
Assignee | |
Comment 10•16 years ago
|
||
OK, I removed the three remaining dirs as well now. Peter: It might be a good idea to start over from the windows packaging lists in the product dirs anyways, like suite/installer/windows/packages
Comment 11•16 years ago
|
||
http://mxr.mozilla.org/seamonkey/source/xpinstall/Makefile.in is still referring to packager and packager/unix BTW: How do we generate .tar.bz2 on unix now?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 12•16 years ago
|
||
(In reply to comment #11) > BTW: How do we generate .tar.bz2 on unix now? > I see. browser/installer mail/installer
![]() |
Assignee | |
Comment 13•16 years ago
|
||
(In reply to comment #11) > http://mxr.mozilla.org/seamonkey/source/xpinstall/Makefile.in > is still referring to packager and packager/unix Thanks. Let's leave this bug FIXED, I'll remove that in bug 387450 along with wizard/ > BTW: How do we generate .tar.bz2 on unix now? Even better than what you found out yourself, use "make package" from the top-level (in the objdir). For installers (Windows-only), use "make installer" from there.
Status: REOPENED → RESOLVED
Closed: 16 years ago → 16 years ago
Resolution: --- → FIXED
Comment 14•16 years ago
|
||
> Even better than what you found out yourself, use "make package" from the > top-level (in the objdir). For installers (Windows-only), use "make installer" > from there. Really? Where is the code for that? You might want to add that to http://developer.mozilla.org/en/docs/Build_and_Install#Installing_Your_Build And you should update http://developer.mozilla.org/en/docs/Building_a_SeaMonkey_package as well.
Comment 15•16 years ago
|
||
> Really? Where is the code for that? To answer my own question: this was added (fairly recently) by bug 377409.
Updated•16 years ago
|
Flags: in-testsuite-
Updated•15 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•