Closed
Bug 534943
Opened 16 years ago
Closed 16 years ago
Clean up the makefiles in lib/ckfw/builtins
Categories
(NSS :: Build, defect)
NSS
Build
Tracking
(Not tracked)
RESOLVED
FIXED
3.12.6
People
(Reporter: wtc, Assigned: wtc)
Details
Attachments
(1 file)
2.25 KB,
patch
|
christophe.ravel.bugs
:
review+
|
Details | Diff | Splinter Review |
The current makefiles in lib/ckfw/builtins work, but they're
using the EXTRA_LIBS/EXTRA_SHARED_LIBS variables differently
from the makefiles for the other NSS shared libraries.
It would be nice for the makefiles for all shared libraries
to be consistent.
The attached patch does the following.
1. Use EXTRA_SHARED_LIBS instead of EXTRA_LIBS to specify
the NSPR shared libraries. EXTRA_SHARED_LIBS is the variable
we use in the makefiles for other NSS shared libraries. See
lib/nss/config.mk, lib/smime/config.mk, and lib/ssl/config.mk.
2. Remove obsolete makefile code for AIX 4.1 and 4.2.
3. Remove commented-out code.
Attachment #417742 -
Flags: review?(christophe.ravel.bugs)
Comment 1•16 years ago
|
||
Comment on attachment 417742 [details] [diff] [review]
Proposed patch
r=Christophe
Attachment #417742 -
Flags: review?(christophe.ravel.bugs) → review+
Assignee | ||
Comment 2•16 years ago
|
||
I checked in the patch on the NSS trunk (NSS 3.12.6).
Checking in Makefile;
/cvsroot/mozilla/security/nss/lib/ckfw/builtins/Makefile,v <-- Makefile
new revision: 1.20; previous revision: 1.19
done
Checking in manifest.mn;
/cvsroot/mozilla/security/nss/lib/ckfw/builtins/manifest.mn,v <-- manifest.mn
new revision: 1.11; previous revision: 1.10
done
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.12.6
You need to log in
before you can comment on or make changes to this bug.
Description
•