Closed Bug 515755 Opened 16 years ago Closed 15 years ago

fix WINCE in-source-dir builds

Categories

(Firefox Build System :: General, defect)

ARM
Windows CE
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: vlad, Assigned: vlad)

Details

Attachments

(1 file, 1 obsolete file)

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)
Attached patch better diffSplinter Review
better diff
Assignee: nobody → vladimir
Attachment #399808 - Attachment is obsolete: true
Attachment #399847 - Flags: review?(bugmail)
Attachment #399808 - Flags: review?(bugmail)
Comment on attachment 399847 [details] [diff] [review] better diff looks good (and much easier to read)
Attachment #399847 - Flags: review?(bugmail) → review+
Flags: in-testsuite-
Version: unspecified → Trunk
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
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: