Closed
Bug 311074
Opened 19 years ago
Closed 19 years ago
NSS build fails when building from readonly sources
Categories
(NSS :: Build, defect)
NSS
Build
Tracking
(Not tracked)
RESOLVED
FIXED
3.11
People
(Reporter: benjamin, Assigned: wtc)
References
Details
Attachments
(2 files, 1 obsolete file)
713 bytes,
patch
|
Details | Diff | Splinter Review | |
7.43 KB,
patch
|
rrelyea
:
review+
|
Details | Diff | Splinter Review |
When I'm building from readonly sources, the NSS build fails on the "clean" target of coreconf/nsinstall because it tries to delete "nfspwd". I'm not really sure why we have both "nfspwd" and "nfspwd.pl" in the tree, but the clean target should definitely not be removing source files.
Reporter | ||
Comment 1•19 years ago
|
||
Attachment #198459 -
Flags: review?(wtchang)
Assignee | ||
Comment 2•19 years ago
|
||
Comment on attachment 198459 [details] [diff] [review] Don't do the weird nfspwd thing The definition of PLSRCS should also be removed.
Attachment #198459 -
Flags: review?(wtchang) → review+
Reporter | ||
Comment 3•19 years ago
|
||
WTC, can you checkin the appropriate fix? I'm not sure what of trunk/branches this belongs on (or even how the NSS branching system works).
Assignee | ||
Comment 4•19 years ago
|
||
This patch is what I checked in on the NSS trunk. This change will eventually appear on the Mozilla trunk (estimate: in a few weeks). Checking in Makefile; /cvsroot/mozilla/security/coreconf/nsinstall/Makefile,v <-- Makefile new revision: 1.10; previous revision: 1.9 done
Attachment #198459 -
Attachment is obsolete: true
Assignee | ||
Comment 5•19 years ago
|
||
nfspwd.pl and nfspwd are Perl scripts that only worked on the old Netscape intranet. Since that network no longer existed, these scripts are obsolete now and should be cvs removed. In makefiles nfspwd should be replaced by the regular pwd command.
Attachment #198639 -
Flags: review?(rrelyea)
Comment 6•19 years ago
|
||
Comment on attachment 198639 [details] [diff] [review] cvs remove the obsolete nfspwd Perl script r+ relyea
Attachment #198639 -
Flags: review?(rrelyea) → review+
Assignee | ||
Comment 7•19 years ago
|
||
nfspwd and nfspwd.pl removed from CVS. Checking in OS2.mk; /cvsroot/mozilla/security/coreconf/OS2.mk,v <-- OS2.mk new revision: 1.24; previous revision: 1.23 done Checking in command.mk; /cvsroot/mozilla/security/coreconf/command.mk,v <-- command.mk new revision: 1.8; previous revision: 1.7 done Checking in rules.mk; /cvsroot/mozilla/security/coreconf/rules.mk,v <-- rules.mk new revision: 1.66; previous revision: 1.65 done Removing nsinstall/nfspwd; /cvsroot/mozilla/security/coreconf/nsinstall/nfspwd,v <-- nfspwd new revision: delete; previous revision: 1.2 done Removing nsinstall/nfspwd.pl; /cvsroot/mozilla/security/coreconf/nsinstall/nfspwd.pl,v <-- nfspwd.pl new revision: delete; previous revision: 1.2 done
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.11
You need to log in
before you can comment on or make changes to this bug.
Description
•