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)
NEW
Future
People
(Reporter: wtc, Unassigned)
Details
Attachments
(2 files)
653 bytes,
patch
|
Details | Diff | Splinter Review | |
1.01 KB,
patch
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → 4.2
Reporter | ||
Comment 1•24 years ago
|
||
Reporter | ||
Comment 2•24 years ago
|
||
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.
Reporter | ||
Comment 3•24 years ago
|
||
Reporter | ||
Comment 4•24 years ago
|
||
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
Reporter | ||
Comment 5•23 years ago
|
||
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 → ---
Comment 6•22 years ago
|
||
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)
Reporter | ||
Updated•19 years ago
|
Status: REOPENED → ASSIGNED
Target Milestone: 4.2 → Future
Updated•18 years ago
|
QA Contact: wtchang → nspr
Comment 7•2 years ago
|
||
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
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•