Closed
Bug 328131
Opened 19 years ago
Closed 1 year ago
NSS cross-compile target cleanup
Categories
(NSS :: Build, defect, P3)
NSS
Build
Tracking
(Not tracked)
RESOLVED
INACTIVE
People
(Reporter: mark, Unassigned)
References
(Blocks 1 open bug)
Details
Wan-Teh pointed out in bug 322578 comment 57:
> Looking at mozilla/security/coreconf/Linux.mk, I
> think that if the cross compilation target is Linux,
> we should override OS_TEST instead of CPU_ARCH. Don't
> you think? On Linux, NSS only uses $(CPU_ARCH) in
> CPU_TAG, which is a component of NSS's OBJDIR_NAME
> and is just a name.
For coreconf-based projects in general, OS_TEST seems to be the right variable to override for a same-OS-different-CPU cross compile, except for Windows, but as he pointed out to me many months ago, cross support in NSS is a little hacky anyway. At present, security/manager doesn't set OS_TEST but CPU_ARCH to drive an NSS cross build. As for Linux, OS_TEST is definitely the variable that should be overridden for cross-compilation. I suspect the same may be true for other Unixish systems, but because security/manager was overriding CPU_ARCH, some of the coreconf makefrags may have since been perverted to support that. (I just perverted the Darwin makefrag in such a way for bug 322578, but hey, there was CPU_ARCH precedent in that file!)
What are the supported NSS cross build combinations? Let's figure out which targets this needs to be cleaned up for.
Comment 1•19 years ago
|
||
Doug, do you still care about cross-compiling NSS,
especially with Windows CE as the target?
If not, please remove yourself from the cc list.
OS: MacOS X → All
Hardware: Macintosh → All
Comment 2•19 years ago
|
||
i would love to have this happen. When Windows CE devices are more robust or there is a demand for more advanced crypto features, I would like to start using NSS again
Updated•19 years ago
|
Assignee: wtchang → nobody
QA Contact: wtchang → build
Updated•18 years ago
|
Priority: -- → P3
Updated•2 years ago
|
Severity: normal → S3
Updated•1 year ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•