Closed
Bug 515755
Opened 16 years ago
Closed 15 years ago
fix WINCE in-source-dir builds
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: vlad, Assigned: vlad)
Details
Attachments
(1 file, 1 obsolete file)
|
5.74 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
I don't advocate in-source-dir builds, but our tinderboxes sometimes do this.
Right now, build/wince/tools has both a Makefile and a Makefile.in, with Makefile.in doing an include $(topsrcdir)/build/wince/tools/Makefile. After configure, the Makefile gets clobbered, and it puts make in an infinite include loop, leading to a cryptic make.exe error message (can't allocate space for cygwin heap).
The solution is to name that Makefile Makefile.standalone, but the tricky thing is that we need to rebuild the tools a second time, since the tools that configure produces are different than the tools that are used for the build (configure's tools don't force-link the shunt, since the shunt isn't built yet at that point).
Probably the most ideal solution is to use an environment variable to skip linking/including the shunt, and have configure export that, and have it build the same tools; but that can be done as a followup patch.
Attachment #399808 -
Flags: review?(bugmail)
| Assignee | ||
Comment 1•16 years ago
|
||
better diff
Assignee: nobody → vladimir
Attachment #399808 -
Attachment is obsolete: true
Attachment #399847 -
Flags: review?(bugmail)
Attachment #399808 -
Flags: review?(bugmail)
Comment 2•16 years ago
|
||
Comment on attachment 399847 [details] [diff] [review]
better diff
looks good (and much easier to read)
Attachment #399847 -
Flags: review?(bugmail) → review+
Updated•16 years ago
|
Flags: in-testsuite-
Version: unspecified → Trunk
Comment 3•15 years ago
|
||
WinCE/Windows Mobile support has been removed from the main build system, Spidermonkey, mobile installer, in-app updater and so on (see bug 614720, bug 554087 and all their dependants). Until such point where MS decide to release a Windows Phone 7 NDK and the decision is made to port to that platform, this is WONTFIX.
Filter bugmail on WinCEMassWONTFIX.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WONTFIX
Updated•8 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•