prtypes.h always included in a CROSS_COMPILE build

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: ash_mozilla, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; en-us) AppleWebKit/532.3+ (KHTML, like Gecko) Version/4.0.3 Safari/531.9
Build Identifier: 

jscpucfg.cpp always includes prtypes.h in a cross compile build.

Patch coming

Reproducible: Always
(Reporter)

Comment 1

8 years ago
Created attachment 407068 [details] [diff] [review]
Simple patch.
(Reporter)

Updated

8 years ago
Blocks: 523964
I don't think we want this exact patch because prtypes.h is what defines IS_BIG_ENDIAN.
(Reporter)

Comment 3

8 years ago
Ahh, this makes a bit more sense. So some form of autoconf test to see if nspr exists for the cross-compile target might be the way to go?
(In reply to comment #3)
> Ahh, this makes a bit more sense. So some form of autoconf test to see if nspr
> exists for the cross-compile target might be the way to go?

No, the problem is that even if you're building with NSPR, jscpucfg is built with the -host- compiler, since it's going to generate a .h file.  So the nspr prtypes.h isn't useful.
Status: UNCONFIRMED → NEW
Ever confirmed: true
No longer blocks: 523964
bug 694499 removed jscpucfg.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Depends on: 694499
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.