Open Bug 63019 Opened 24 years ago Updated 2 years ago

NSPR build system should only build shared libraries.

Categories

(NSPR :: NSPR, defect, P3)

Tracking

(Not tracked)

Future

People

(Reporter: wtc, Unassigned)

Details

Attachments

(2 files)

We have been making the statement that we
only support NSPR shared libraries, but
we haven't got around to removing static
libraries from our build system, so static
libraries are still being built.

We should remove static libraries from our
build system.  This should be done in the
next minor release (4.2).  If a platform
does not have shared libraries, we will
still build and support static libraries
on that platform.
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → 4.2
Attached patch Proposed fix.Splinter Review
I verified the patch works on the following platforms.
1. Solaris, classic build system.  Static libraries
   don't get built.
2. Solaris, with makefile modifications to simulate a
   platform without shared libraries, classic build
   system.  Static libraries get built correctly.
3. Linux, with autoconf.  Static libraries don't get
   built.
4. Windows 2000 (classic build system).  Static
   libraries don't get built.

I would appreciate a code review before I check
it in.
Checked in the fix on the main trunk.
/cvsroot/mozilla/nsprpub/config/rules.mk, revision 3.34
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Sigh.  I found that NSS is still using NSPR static libraries.
So I had to back out my fix.  I will put the fix back in when
the NSS bug is fixed.

Reopened the bug.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
was the NSS bug fixed? ie. what is the current status on this? Is this 
something that is still an issue?

(trying to help focus)
Status: REOPENED → ASSIGNED
Target Milestone: 4.2 → Future
QA Contact: wtchang → nspr

The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.

Assignee: wtc → nobody
Status: ASSIGNED → NEW
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: