Closed
Bug 849175
Opened 11 years ago
Closed 11 years ago
MozillaBuild uses wrong path for MSVC11 Express
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 903945
People
(Reporter: Mitch, Assigned: Mitch)
Details
Attachments
(1 file)
1.08 KB,
patch
|
ted
:
review-
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•11 years ago
|
||
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 2•11 years ago
|
||
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 3•11 years ago
|
||
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-
Comment 4•11 years ago
|
||
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.
Assignee | ||
Comment 5•11 years ago
|
||
Bug 903945 has active work on it so I'm resolving this as a duplicate.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•