Fix msvs-ninja OutputDirectory path.

test/lib/TestGyp.py

Index: test/lib/TestGyp.py
===================================================================
--- test/lib/TestGyp.py	(revision 1924)
+++ test/lib/TestGyp.py	(working copy)
@@ -78,6 +78,7 @@
   configuration and to run executables generated by those builds.
   """
 
+  formats = []
   build_tool = None
   build_tool_list = []
 
@@ -113,6 +114,8 @@
     self.gyp = os.path.abspath(gyp)
     self.no_parallel = False
 
+    self.formats = [self.format]
+
     self.initialize_build_tool()
 
     kw.setdefault('match', TestCommon.match_exact)
@@ -130,10 +133,11 @@
 
     super(TestGypBase, self).__init__(*args, **kw)
 
+    real_format = self.format.split('-')[-1]
     excluded_formats = set([f for f in formats if f[0] == '!'])
     included_formats = set(formats) - excluded_formats
-    if ('!'+self.format in excluded_formats or
-        included_formats and self.format not in included_formats):
+    if ('!'+real_format in excluded_formats or
+        included_formats and real_format not in included_formats):
       msg = 'Invalid test for %r format; skipping test.\n'
       self.skip_test(msg % self.format)
 
@@ -272,9 +276,13 @@
 
     # TODO:  --depth=. works around Chromium-specific tree climbing.
     depth = kw.pop('depth', '.')
-    run_args = ['--depth='+depth, '--format='+self.format, gyp_file]
+    run_args = ['--depth='+depth]
+    run_args.extend(['--format='+f for f in self.formats]);
+    run_args.append(gyp_file)
     if self.no_parallel:
       run_args += ['--no-parallel']
+    # TODO: if extra_args contains a '--build' flag
+    # we really want that to only apply to the last format (self.format).
     run_args.extend(self.extra_args)
     run_args.extend(args)
     return self.run(program=self.gyp, arguments=run_args, **kw)
@@ -1002,6 +1010,120 @@
     return self.workpath(*result)
 
 
+def FindMSBuildInstallation():

[bungeman-chromium, 2014/05/30 22:18:09]

This is the better way to find MSBuild, not integrated with
FindVisualStudioInstallation.

+  """Returns path to MSBuild for GYP_MSVS_VERSION, msvs_version,
+  or latest available.
+
+  Looks in the registry to find install location of MSBuild.
+  MSBuild before v4.0 will not build c++ projects, so only use newer versions.
+  """
+  import _winreg
+  def get_reg_key(base, key):
+    try:
+      result = _winreg.OpenKey(base, key, 0,
+                               _winreg.KEY_READ | _winreg.KEY_WOW64_32KEY)
+    except WindowsError:
+      result = None
+    return result
+
+  def get_reg_key_value(key, value):
+    try:
+      result, _ = _winreg.QueryValueEx(key, value)
+    except WindowsError:
+      result = None
+    return result
+
+  msbuild_basekey = get_reg_key(_winreg.HKEY_LOCAL_MACHINE,
+                                'SOFTWARE\Microsoft\MSBuild\ToolsVersions')
+  if not msbuild_basekey:
+    print 'Error: could not find MSBuild base registry entry'
+    sys.exit(1)
+
+  msvs_to_msbuild = {
+      '2013': r'12.0',
+      '2012': r'4.0', #really v4.0.30319 which comes with .NET 4.5
+      '2010': r'4.0'}
+
+  msvs_version = 'auto'
+  for flag in (f for f in sys.argv if f.startswith('msvs_version=')):
+    msvs_version = flag.split('=')[-1]
+  msvs_version = os.environ.get('GYP_MSVS_VERSION', msvs_version)
+
+  msbuild_key = None
+  if msvs_version in msvs_to_msbuild:
+    msbuild_version = msvs_to_msbuild[msvs_version]
+    msbuild_key = get_reg_key(msbuild_basekey, msbuild_version)
+    if not msbuild_key:
+      print ('Warning: Environment variable GYP_MSVS_VERSION specifies "%s" '
+             'but corresponding MSBuild "%s" was not found.' %
+             (msvs_version, msbuild_version))
+  if not msbuild_key:
+    for msvs_version in sorted(msvs_to_msbuild, reverse=True):
+      msbuild_version = msvs_to_msbuild[msvs_version]
+      msbuild_key = get_reg_key(msbuild_basekey, msbuild_version)
+  if not msbuild_key:
+    print 'Error: could not find MSBuild registry entry'
+    sys.exit(1)
+
+  msbuild_path = get_reg_key_value(msbuild_key, 'MSBuildToolsPath')
+  if not msbuild_path:
+    print 'Error: could not get MSBuild registry entry value'
+    sys.exit(1)
+
+  return os.path.join(msbuild_path, 'MSBuild.exe')
+
+
+class TestGypMSVSNinja(TestGypNinja):
+  """
+  Subclass for testing the GYP Visual Studio Ninja generator.
+  """
+  format = 'msvs-ninja'
+
+  def initialize_build_tool(self):
+    super(TestGypMSVSNinja, self).initialize_build_tool()
+    self.msbuild_path = FindMSBuildInstallation()
+    # When using '--build', make sure ninja is first in the format list.
+    self.formats.insert(0, 'ninja')
+
+  def build(self, gyp_file, target=None, rebuild=False, clean=False, **kw):
+    """
+    Runs a Visual Studio build using the configuration generated
+    from the specified gyp_file.
+    """
+    arguments = kw.get('arguments', [])[:]
+    if target in (None, self.ALL, self.DEFAULT):
+      # Note: the Visual Studio generator doesn't add an explicit 'all' target.
+      # This will build each project. This will work if projects are hermetic,
+      # but may fail if they are not (a project may run more than once).
+      # It would be nice to supply an all.metaproj for MSBuild.
+      arguments.extend([gyp_file.replace('.gyp', '.sln')])
+    else:
+      # MSBuild documentation claims that one can specify a sln but then build a
+      # project target like 'msbuild a.sln /t:proj:target' but this format only
+      # supports 'Clean', 'Rebuild', and 'Publish' (with none meaning Default).
+      # This limitation is due to the .sln -> .sln.metaproj conversion.
+      # The ':' is not special, 'proj:target' is a target in the metaproj.
+      arguments.extend([target+'.vcxproj'])
+
+    if clean:
+      build = 'Clean'
+    elif rebuild:
+      build = 'Rebuild'
+    else:
+      build = 'Build'
+    arguments.extend(['/target:'+build])
+    configuration = self.configuration_buildname()
+    config = configuration.split('|')
+    arguments.extend(['/property:Configuration='+config[0]])
+    if len(config) > 1:
+      arguments.extend(['/property:Platform='+config[1]])
+    arguments.extend(['/property:BuildInParallel=false'])
+    arguments.extend(['/verbosity:minimal'])
+
+    kw['arguments'] = arguments
+    return self.run(program=self.msbuild_path, **kw)
+
+
 class TestGypXcode(TestGypBase):
   """
   Subclass for testing the GYP Xcode generator.
@@ -1118,6 +1240,7 @@
   TestGypCMake,
   TestGypMake,
   TestGypMSVS,
+  TestGypMSVSNinja,
   TestGypNinja,
   TestGypXcode,
 ]