358953 - visibility patches for 1.8 branch

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

These are two straggling visibility patches needed to compile the 1.8 branch on Fedora Core 6, but are not needed for the trunk.  This fix relocations that (given WRAP_SYSTEM_INCLUDES works on FC6) cause linker errors with x86_64 or prevent running under SELinux with i686.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patches

The first patch changes the extern declaration in nsNativeComponentLoader.cpp (which no longer exists on the trunk) to match the definition in nsComponentManager.cpp by adding NS_COM.

The second patch already landed on the trunk as part of attachment 222874 [details] [diff] [review] of bug 305949.

Comment 2

[Benjamin Smedberg]

Comment on attachment 244243 [details] [diff] [review]
patches

Why nsComponentManagerLog? I don't see that being used outside of libxpcom_core.so.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

We could easily go the other way, I suppose, and remove the NS_COM at the definition.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patches

This removes the other NS_COM rather than adding a new one.  This part can therefore also land on the trunk.

I also added a change to xpcom/reflect/xptcall/src/md/unix/Makefile.in .  I'd thought that I'd tested that bug 358931 was sufficient, but it's not.  So I'm pulling the MOZILLA_INTERNAL_API=1 across from the trunk.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Checked in the NS_COM removal to the trunk; the other two pieces are things that I got from the trunk, so marking FIXED.

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 244500 [details] [diff] [review]
patches

This is a low risk patch that will, along with the other dependencies of bug 359275:

* allow Linux distros to take advantage of improved load and runtime performance with WRAP_SYSTEM_HEADERS, other performance optimizations
(prelinking), and security improvements (SELinux)

* make it easier for developers to build on new distributions like Fedora Core
6

Comment 7

[Andrew Schultz]

The nsComponentManager change seems to have broken nye's builds except for the bloat build:
http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTest/1162624800.21811.gz

nye builds with gcc 3.4.4.  The bloat build has visibility completely disabled to get tracemalloc symbols, while the other trees build with visibility half-enabled '-fvisibility-hidden' since they have the gcc bug.

Comment 8

[Robert Kaiser]

My local SeaMonkey builds (gcc 4.1.0) are busted due to this change as well, with the same error as nye tinderbox. As http://lxr.mozilla.org/mozilla/search?string=nsComponentManagerLog shows, this is also used in nsRegistry.cpp (libxpcom_compat_c.so) and that's where it's failing...

Comment 9

[Robert Kaiser]

REOPENing as this broke compiling SeaMonkey trunk on modern enough compilers.

Comment 10

[Robert Kaiser]

Attachment: bustage fix for SeaMonkey trunk

Reverting this change makes SeaMonkey trunk compile again. I don't know the code and don't know if this is the correct way to fix it, but at least this fixes the current bustage. I'll leave it to more competent people to fix the real issue if needed.

Comment 11

[Benjamin Smedberg]

That's bug 359512, and we should solve it by removing the logging from nsRegistry.

Comment 12

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 244683 [details] [diff] [review]
bustage fix for SeaMonkey trunk

I backed out the change, and you guys can discuss what to do with it on the trunk in bug 359512.

I'll go with my original approach for the 1.8 branch.

Comment 13

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patches

This is a low risk patch that will, along with the other dependencies of bug
359275:

* allow Linux distros to take advantage of improved load and runtime
performance with WRAP_SYSTEM_HEADERS, other performance optimizations
(prelinking), and security improvements (SELinux)

* make it easier for developers to build on new distributions like Fedora Core
6

Comment 14

[Daniel Veditz [:dveditz]]

Comment on attachment 244688 [details] [diff] [review]
patches

approved for 1.8/1.8.0 branches, a=dveditz

Comment 15

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Checked in to MOZILLA_1_8_BRANCH and MOZILLA_1_8_0_BRANCH.