Closed
Bug 403994
Opened 17 years ago
Closed 17 years ago
Don't install nss headers when system-nss is used
Categories
(Core :: Security: PSM, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9beta2
People
(Reporter: fta+bugzilla, Assigned: fta+bugzilla)
References
Details
Attachments
(1 file)
1.03 KB,
patch
|
rrelyea
:
review+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b2pre) Gecko/2007111523 Minefield/3.0b2pre Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b2pre) Gecko/2007111523 Minefield/3.0b2pre bug 389872 started to install nss headers inconditionally. It makes the build fail on some platforms (firefox, xulrunner and seamonkey). make[4]: Entering directory `/build/buildd/seamonkey-2.0/security/manager' /build/buildd/seamonkey-2.0/config/nsinstall -D ../../dist/include/nss (cd ../../dist/public/nss && tar -cvhf - .) | \ (cd ../../dist/include/nss && tar -xf -) cd: 2: can't cd to ../../dist/public/nss tar: This does not look like a tar archive tar: Error exit delayed from previous errors make[4]: *** [libs] Error 2 make[4]: Leaving directory `/build/buildd/seamonkey-2.0/security/manager' Strangely, on other platforms, it's not fatal: (cd ../../dist/public/nss && tar -cvhf - .) | \ (cd ../../dist/include/nss && tar -xf -) cd: 2: can't cd to ../../dist/public/nss (cd ../../dist/public/nss && tar -cvhf - .) | \ (cd ../../dist/sdk/include && tar -xf -) cd: 2: can't cd to ../../dist/public/nss /usr/bin/make -C boot libs ... in both case, /bin/sh is in fact dash, but not the same version. Reproducible: Always Steps to Reproduce: 1. 2. 3.
Assignee | ||
Comment 1•17 years ago
|
||
Assignee | ||
Updated•17 years ago
|
Summary: Don't install nss headers when sysytem-nss is used → Don't install nss headers when system-nss is used
Updated•17 years ago
|
Assignee: nobody → fta+bugzilla
Component: Build Config → Security: PSM
Product: Firefox → Core
QA Contact: build.config → psm
Version: unspecified → Trunk
Assignee | ||
Updated•17 years ago
|
Attachment #288937 -
Flags: review?
Updated•17 years ago
|
Attachment #288937 -
Flags: review? → review?(rrelyea)
Comment 2•17 years ago
|
||
Comment on attachment 288937 [details] [diff] [review] protect install by MOZ_NATIVE_NSS ifdef Yes, you don't want to bring in these headers if you are using system NSS (which may have a newer version of NSS than included in these source file).
Attachment #288937 -
Flags: review?(rrelyea) → review+
Comment 3•17 years ago
|
||
Comment on attachment 288937 [details] [diff] [review] protect install by MOZ_NATIVE_NSS ifdef Simple build fix for Linux distros who wish to use system NSS instead of the shipped NSS.
Attachment #288937 -
Flags: approval1.9?
Updated•17 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Updated•17 years ago
|
Attachment #288937 -
Flags: approval1.9? → approval1.9+
Comment 4•17 years ago
|
||
Checking in security/manager/Makefile.in; /cvsroot/mozilla/security/manager/Makefile.in,v <-- Makefile.in new revision: 1.79; previous revision: 1.78 done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M10
You need to log in
before you can comment on or make changes to this bug.
Description
•