Closed
Bug 1139135
Opened 11 years ago
Closed 6 years ago
Feature request: Mach should fail early on Win32.
Categories
(Firefox Build System :: General, enhancement)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: mhoye, Unassigned)
Details
(Keywords: good-first-bug)
Per Bug 1137346 we can no longer build on 32 bit Windows (possibly single-CPU?) Windows systems.
I propose that as soon as mach figures out it's running on Win32 - ideally as close to the start of the build process as possible - that it bail out immediately with an explanation rather than wait (possibly for hours) until we fail at the linker stage.
Comment 1•11 years ago
|
||
I can still build on win32 Windows 7 with dual cpus. Please don't totally kill the possibility of building on win32.
| Reporter | ||
Comment 2•11 years ago
|
||
OK, add a --force-win32-build option too maybe
Comment 3•11 years ago
|
||
The proper place for this is a configure check.
Component: mach → Build Config
Comment 4•11 years ago
|
||
We could stick a test in here:
https://hg.mozilla.org/mozilla-central/annotate/0189941a3fd5/configure.in#l1866 (where we're already checking that the host system is windows), something like:
if test "$PROCESSOR_ARCHITECTURE" = "x86" -a -z "$PROCESSOR_ARCHITEW6432"; then
AC_MSG_ERROR([Building on 32-bit Windows is not supported.])
fi
| Reporter | ||
Comment 5•11 years ago
|
||
OK, I'm going to call this a good first bug and shop it around.
Whiteboard: [good first bug]
Updated•8 years ago
|
Product: Core → Firefox Build System
Updated•6 years ago
|
Keywords: good-first-bug
Whiteboard: [good first bug]
Comment 6•6 years ago
|
||
Hello, Is this bug still valid? Which file should I put the check in ?
Flags: needinfo?(mhoye)
| Reporter | ||
Comment 7•6 years ago
|
||
Sorry, Jayati - this bug should have been closed some time ago. It dates to a time when it was not possible to build on Win32 because of a linker memory problem, but that bug has been resolved. Please find us on Matrix, so we can find you something to work on though! Thank you for your interest.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(mhoye)
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•