The current script assumes that you've installed the compiler. We should check the actual subkey that shows that it's installed.
Created attachment 458308 [details] [diff] [review] Possible patch I noticed this because mozillabuild couldn't correctly find my compiler after I had upgraded to the Vista SDK 6.0A. (In the meantime I worked around the problem by creating a junction point back to the actual install location). I should try installing the Windows 7 SDK compiler (VC9SP1EXPRESS) to see if I can write detection for it too.
Attachment #458308 - Flags: review?(ted.mielczarek)
Unlike the Vista SDK, the Windows 7 SDK puts the 32-bit compiler in the same place that VS2008 Express does, but instead of providing amd64\vcvarsamd64.bat (which I assume is provided by VS2008 Professional) it provides a choice of either vcvars64.bat (which is a true 64-bit compiler) or vcvarsx86_amd64.bat (which is a 32-bit compiler that produces 64-bit executables).
Attachment #458308 - Flags: review?(ted.mielczarek) → review+
Pushed with wrong bug number in comment: http://hg.mozilla.org/mozilla-build/rev/fb0a68519dcc
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.