Closed
Bug 320377
Opened 19 years ago
Closed 19 years ago
Cannot run regxpcom anymore on Sparc/Solaris (dynamic linking not supported)
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: InvisibleSmiley, Unassigned)
Details
(Keywords: fixed1.8.0.1, fixed1.8.1)
Attachments
(4 files, 2 obsolete files)
431 bytes,
patch
|
zhayupeng
:
review+
|
Details | Diff | Splinter Review |
431 bytes,
patch
|
zhayupeng
:
review+
|
Details | Diff | Splinter Review |
431 bytes,
patch
|
Details | Diff | Splinter Review | |
431 bytes,
patch
|
dveditz
:
approval1.8.0.1+
dveditz
:
approval1.8.1+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051206 SeaMonkey/1.5a Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051206 SeaMonkey/1.5a Sparc/Solaris 5.8, building from source, Compiler: cc: Sun WorkShop 6 update 2 C 5.3 Patch 111679-11 2003/04/02 With Firefox 1.5 and current SeaMonkey nightly builds from trunk, I cannot run regxpcom any more. It used to work with all Firefox 1.x versions and older Mozilla versions (unfortunately, I cannot tell exactly when it broke). Reproducible: Always Steps to Reproduce: 1. cd to application directory 2. run ./run-mozilla.sh ./regxpcom Actual Results: Output: XPCOM glue dynamic linking is not implemented on this platform!Can not initialize XPCOM Glue Can not aquire component registrar Expected Results: It should work
Reporter | ||
Updated•19 years ago
|
OS: Other → Solaris
Hardware: Other → Sun
Comment 1•19 years ago
|
||
http://lxr.mozilla.org/mozilla/source/xpcom/glue/standalone/Makefile.in#71 You either need to add Solaris to one of the existing conditions (dlopen?) or write a dynamic linking stub for this platform (use of NSPR is not allowed in this code due to the way the standalone glue needs to work).
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•19 years ago
|
Assignee: dougt → nobody
Reporter | ||
Comment 2•19 years ago
|
||
(In reply to comment #1) > http://lxr.mozilla.org/mozilla/source/xpcom/glue/standalone/Makefile.in#71 > > You either need to add Solaris to one of the existing conditions (dlopen?) or > write a dynamic linking stub for this platform (use of NSPR is not allowed in > this code due to the way the standalone glue needs to work). From config/autoconf.mk I can tell that OS_ARCH is SunOS here. Adding this to the list in line 64 seems to do the trick (after a recompile/reinstall, regxpcom is working again). If someone can verify this I'd suggest checking this in on trunk and all relevant branches (SM 1.0, FF 1.5, TB 1.5; maybe more) ASAP.
Comment 3•19 years ago
|
||
Please provide a patch.
Reporter | ||
Comment 4•19 years ago
|
||
Reporter | ||
Comment 5•19 years ago
|
||
Reporter | ||
Comment 6•19 years ago
|
||
Don't know whether SeaMonkey trunk and Firefox trunk are the same; maybe FF trunk needs yet another patch. Thunderbird should also be taken care of, but I have no source for the 1.5 branch. There may be even more branches needing the fix (Sunbird?) but I don't know about that either.
Comment 7•19 years ago
|
||
One patch is sufficient, it will end up on three trees (trunk, 1.8, and 1.8.0).
Updated•19 years ago
|
Attachment #205974 -
Flags: review?(pete.zha)
Comment on attachment 205974 [details] [diff] [review] SeaMonkey trunk patch >--- xpcom/glue/standalone/Makefile Thu Dec 15 16:48:50 2005 >+++ xpcom/glue/standalone/Makefile.new Thu Dec 15 16:44:37 2005 You need to modify Makefile.in instead of Makefile.
Reporter | ||
Comment 9•19 years ago
|
||
Attachment #205974 -
Attachment is obsolete: true
Attachment #205974 -
Flags: review?(pete.zha)
Reporter | ||
Comment 10•19 years ago
|
||
Attachment #205975 -
Attachment is obsolete: true
Comment 11•19 years ago
|
||
Comment on attachment 206072 [details] [diff] [review] SeaMonkey trunk Makefile.in patch r=pete
Attachment #206072 -
Flags: review+
Comment 12•19 years ago
|
||
Comment on attachment 206073 [details] [diff] [review] Firefox 1.5 Makefile.in patch r=pete if appropriate
Attachment #206073 -
Flags: review+
Comment 13•19 years ago
|
||
Fixed on trunk. (I put SunOS in it's proper alphabetical order)
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Attachment #206072 -
Flags: approval1.8.1?
Attachment #206072 -
Flags: approval1.8.0.1?
Comment 14•19 years ago
|
||
From Comment 13 - can we get a patch with the proper ordering? Thanks!
Reporter | ||
Comment 15•19 years ago
|
||
Not checking Obsoletes since other patch has review
Reporter | ||
Comment 16•19 years ago
|
||
Not checking Obsoletes since other patch has review
Comment 17•19 years ago
|
||
Comment on attachment 206375 [details] [diff] [review] SeaMonkey trunk patch with proper ordering a=dveditz for drivers
Attachment #206375 -
Flags: approval1.8.1+
Attachment #206375 -
Flags: approval1.8.0.1+
Updated•19 years ago
|
Attachment #206072 -
Flags: approval1.8.1?
Attachment #206072 -
Flags: approval1.8.0.1?
You need to log in
before you can comment on or make changes to this bug.
Description
•