haniceboy / gyp

Automatically exported from code.google.com/p/gyp
0 stars 0 forks source link

Gyp will not auto-detect Visual Studio 2013 #377

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Run Gyp on a Windows computer with only VS 2013 or VC++ Express for Desktop 
2013.

What is the expected output? What do you see instead?
VS 2013 to be detected

Please provide any additional information below.

Note below that 'auto' does not include '12.0' in the list of versions to 
search.

version_map = {
    'auto': ('10.0', '9.0', '8.0', '11.0'),
    '2005': ('8.0',),
    '2005e': ('8.0',),
    '2008': ('9.0',),
    '2008e': ('9.0',),
    '2010': ('10.0',),
    '2010e': ('10.0',),
    '2012': ('11.0',),
    '2012e': ('11.0',),
    '2013': ('12.0',),
    '2013e': ('12.0',),
}

Original issue reported on code.google.com by j...@hikinghomeschoolers.org on 31 Oct 2013 at 3:58

GoogleCodeExporter commented 9 years ago
I'd suggest that '12.0' be ranked first in the list, prior to '10.0' given the 
emphasis on improving the C++ compiler and move towards standards based C++ in 
the 2013 release.

Original comment by j...@hikinghomeschoolers.org on 31 Oct 2013 at 4:02

GoogleCodeExporter commented 9 years ago
https://codereview.chromium.org/55493003/patch/1/10001

Original comment by j...@hikinghomeschoolers.org on 31 Oct 2013 at 11:31

GoogleCodeExporter commented 9 years ago
Fixed in R1776.  Thank you!

Original comment by j...@hikinghomeschoolers.org on 5 Nov 2013 at 4:27

GoogleCodeExporter commented 9 years ago
One more issue, does not detect the new executable name in Visual Studio 
Express for Windows Desktop 2013, as it's been renamed from vcexpress.exe to 
wdexpress.exe.  Modified the logic to validate the file *express.exe instead.  
CR up at:
https://codereview.chromium.org/68263002

Original comment by j...@hikinghomeschoolers.org on 11 Nov 2013 at 1:17

GoogleCodeExporter commented 9 years ago
Should be fixed in r1781.

Original comment by yukawa@chromium.org on 23 Nov 2013 at 12:00