Closed Bug 92222 Opened 24 years ago Closed 23 years ago

xpinstall/wizard doesnt build on IRIX

Categories

(SeaMonkey :: Installer, defect)

SGI
IRIX
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: johnv, Assigned: dveditz)

References

Details

Attachments

(2 files)

xpinstall/wizard does not build on IRIX
To verify the patch I have downloaded the linux installer and dropped the mips dist/bin/mozilla-installer-bin over the i686 binary. This downloaded the xpi files correctly, however it was unable to install mozilla; I presume due to the xpi files being i686?
QA Contact: gemal → gbush
Thanks for the patch! Yes, the install would fail because of binary incompatibility. Once the .xpi files are downloaded the first xpcom.xpi is expanded and used as the install engine for the rest. Once at that point you'd need IRIX bits to proceed.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: patch
Reassigning to syd for unix installer issues. Is the first patch really the right thing? rather than multiple ugly #ifdef IRIX sections does the lack of socklen_t mean there's a missing #include on IRIX? Or could we typedef the equivalence once at the top instead?
Assignee: ssu → syd
I can not find an IRIX system header with socklen_t in it, however you are right, I should have used the typedef already in place for Win/Solaris.
Blocks: 92324
r=dveditz@netscape.com I'll try to get the required sr= for this and get it in.
Assignee: syd → dveditz
Keywords: nsBranch
Blocks: 73732
Seems like this has all the reviews it needs. Can you mark this in the attachment section and change keyword to nsbranch+? After that, send an email to pdt2@netscape.com so we can discuss this for possible eMojo branch checkin.
Cindy, I wonder if we need this on the branch? Would the trunk suffice for the Irix platform support?
How this got nominated nsbranch is mystifying. nsbranch-
Keywords: nsbranchnsbranch-
Fix checked in, thanks. Sorry for the delay, missed kin's sr=
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
verified code fix
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: