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
•