Closed
Bug 475027
Opened 16 years ago
Closed 16 years ago
cross-compiling to mingw broken
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: benjamin, Assigned: benjamin)
References
Details
(Keywords: fixed1.9.1, regression)
Attachments
(1 file, 1 obsolete file)
1.36 KB,
patch
|
crowderbt
:
review+
|
Details | Diff | Splinter Review |
Neil indicated that cross-compiling to mingw was broken by bug 269538. I don't have a mingw-cross environment to check with, but here are some thoughts and theories: 1) are you hitting this block in js/src/configure.in at all? http://mxr.mozilla.org/mozilla-central/source/js/src/configure.in#2556 If not, we need to figure out why: only MSVC should be setting SKIP_COMPILER_CHECKS, here: http://mxr.mozilla.org/mozilla-central/source/js/src/configure.in#571 2) The CROSS_COMPILE check in jstypes.h is silly. We should just check for MSC_VER and otherwise use jsautocfg.h. I'll attach a patch shortly.
Assignee | ||
Comment 1•16 years ago
|
||
Comment 2•16 years ago
|
||
Comment on attachment 358422 [details] [diff] [review] Make inclusion of jscpucfg.h/jsautocfg.h more sane, rev. 1 You could change the comment on the #inclusion of jscpucfg.h not to mention Mac.
Assignee | ||
Comment 3•16 years ago
|
||
What Neil is actually doing is cross-compiling from FreeBSD->Windows using *MSVC* via Wine. This patch makes it so that we only use jscpucfg.h for MSVC, and use jsautocfg.h in all other configurations.
Assignee: nobody → benjamin
Attachment #358422 -
Attachment is obsolete: true
Attachment #358455 -
Flags: review?(crowder)
Updated•16 years ago
|
Attachment #358455 -
Flags: review?(crowder) → review+
Assignee | ||
Comment 4•16 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/b4ddaaf84af5
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•16 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/558f032401e5 Pushed to 1.9.1 as a part of the sequence of patches for bug 470071
Flags: blocking1.9.1+
Keywords: fixed1.9.1,
regression
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•