Closed Bug 291993 Opened 18 years ago Closed 18 years ago

Build using configure on Solaris fails to find NSPR libs


(Directory :: LDAP C SDK, defect)

Not set


(Not tracked)



(Reporter: richm, Assigned: richm)



(1 file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.2) Gecko/20040805 Netscape/7.2
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.2) Gecko/20040805 Netscape/7.2

This only happens with libprldap and libssldap.  It uses -L$(dist_libdir) to
find the NSPR libs but they are actually in dist/PLATFORM/lib.  I've changed it
to use -L$(DIST)/$(RELEASE_OBJDIR_NAME)/lib and it works, but I don't know if
that is the most appropriate macro to use.

Reproducible: Always
Attached file Diffs for fix
Uses $(DIST)/$(RELEASE_OBJDIR_NAME)/lib to find the NSPR shared libs.
Assignee: mcs → richm
Why is this only a problem on Solaris?
I don't know.  I suspect that it will be a problem on other platforms that
require shared libraries to link against when building a shared libraries. 
Another way to do it would be to omit the NSPR libraries when linking  Are they really required?  It seems that linking
does not require the NSS libraries, only the NSPR libraries.  I'm not sure why.
A little more digging reveals that Solaris builds just fine without building in
any dependencies to the NSPR shared libraries.

On Linux, and are built with no run time dependencies.

Other components do build in dependencies.  For example, NSS builds in run time
dependencies to NSPR.

It seems like the right thing to do is to build in run time dependencies to NSPR
and NSS on all platforms, to at least make it consistent.
I'd like to go ahead and check in the fix listed in the diffs, to at least
enable building on Solaris using the configure build method.
Checking in mozilla/directory/c-sdk/ldap/libraries/libprldap/;
/cvsroot/mozilla/directory/c-sdk/ldap/libraries/libprldap/,v  <--
new revision: 5.11; previous revision: 5.10
Checking in mozilla/directory/c-sdk/ldap/libraries/libssldap/;
/cvsroot/mozilla/directory/c-sdk/ldap/libraries/libssldap/,v  <--
new revision: 5.9; previous revision: 5.8
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.