Closed Bug 849175 Opened 11 years ago Closed 11 years ago

MozillaBuild uses wrong path for MSVC11 Express

Categories

(Firefox Build System :: General, defect)

x86_64
Windows 8
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 903945

People

(Reporter: Mitch, Assigned: Mitch)

Details

Attachments

(1 file)

start-msvc11-x64.bat output below:

"Mozilla tools directory: C:\mozilla-build\"
Visual C++ 10 Express directory: C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\
Visual C++ 11 directory: C:\Program Files (x86)\Microsoft Visual Studio 11.0\VC\
Visual C++ 11 Express directory: C:\Program Files (x86)\Microsoft Visual Studio 11.0\VC\
Windows SDK directory: C:\Program Files (x86)\Windows Kits\8.0\
Windows SDK version: 8.0
Using VC 2012 built-in SDK
The system cannot find the path specified.
Mozilla build environment: MSVC version 11.
There doesn't appear to be a native x64 cl.exe with the Express edition, so the only option is to use the 32-bit cross compiler to produce x64 builds.
Attachment #722735 - Flags: review?(gps)
Comment on attachment 722735 [details] [diff] [review]
Use x86 cross compiler for x64 target (patch v1)

I'm not sure what the full implications of this are. Ted should be able to review this.
Attachment #722735 - Flags: review?(gps) → review?(ted)
Comment on attachment 722735 [details] [diff] [review]
Use x86 cross compiler for x64 target (patch v1)

Review of attachment 722735 [details] [diff] [review]:
-----------------------------------------------------------------

This isn't right. You're modifying the path that's supposed to setup VC11 pro, not express. From your log output it looks like we're detecting your Express install as Pro. :-/

Maybe the logic in guess-msvc.bat needs an update instead?
http://hg.mozilla.org/mozilla-build/file/tip/guess-msvc.bat
Attachment #722735 - Flags: review?(ted) → review-
I have VC 2012 Pro installed here now, so if you want to compare notes on what registry keys are present we can do that.
Bug 903945 has active work on it so I'm resolving this as a duplicate.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
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: