Detect debug/release of out\Default from args.gn

chrome/tools/build/win/create_installer_archive.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Script to create Chrome Installer archive.
 
   This script is used to create an archive of all the files required for a
   Chrome install in appropriate directory structure. It reads chrome.release
   file as input, creates chrome.7z archive, compresses setup.exe and
   generates packed_files.txt for mini_installer project.
 
 """
 
 import ConfigParser
 import glob
 import optparse
 import os
+import re
 import shutil
 import subprocess
 import sys
 
 
 ARCHIVE_DIR = "installer_archive"
 
 # suffix to uncompresed full archive file, appended to options.output_name
 ARCHIVE_SUFFIX = ".7z"
 BSDIFF_EXEC = "bsdiff.exe"
@@ skipping 386 matching lines @@
       dest_data = fdest.read()
     if src_data != dest_data:
       # This may still raise if we get here, but this really should almost
       # never happen (it would mean that the contents of e.g. msvcr100d.dll
       # had been changed).
       shutil.copyfile(src, dest)
   else:
     shutil.copyfile(src, dest)
 
 
+# Taken and modified from:
+# third_party\WebKit\Tools\Scripts\webkitpy\layout_tests\port\factory.py
+def _read_configuration_from_gn(build_dir):
+  """Return the configuration to used based on args.gn, if possible."""
+  path = os.path.join(build_dir, 'args.gn')
+  if not os.path.exists(path):
+    path = os.path.join(build_dir, 'toolchain.ninja')
+    if not os.path.exists(path):
+      # This does not appear to be a GN-based build directory, so we don't
+      # know how to interpret it.
+      return None
+
+    # toolchain.ninja exists, but args.gn does not; this can happen when
+    # `gn gen` is run with no --args.
+    return 'Debug'
+
+  args = open(path).read()
+  for l in args.splitlines():
+    m = re.match('^\s*is_debug\s*=\s*false(\s*$|\s*#.*$)', l)

[robertshield, 2016/04/05 23:43:43]

Could you add a comment here that this regex is correct as per
https://chromium.googlesource.com/chromium/src/+/master/tools/gn/docs/referen...
?

[brucedawson, 2016/04/06 00:25:05]

Done.

+    if m:
+      return 'Release'
+
+  # if is_debug is set to anything other than false, or if it
+  # does not exist at all, we should use the default value (True).
+  return 'Debug'
+
+
 # Copy the relevant CRT DLLs to |build_dir|. We copy DLLs from all versions
 # of VS installed to make sure we have the correct CRT version, unused DLLs
 # should not conflict with the others anyways.
 def CopyVisualStudioRuntimeDLLs(target_arch, build_dir):
   is_debug = os.path.basename(build_dir).startswith('Debug')
   if not is_debug and not os.path.basename(build_dir).startswith('Release'):
-    print ("Warning: could not determine build configuration from "
-           "output directory, assuming Release build.")
+    gn_type = _read_configuration_from_gn(build_dir)
+    if gn_type == 'Debug':
+      is_debug = True
+    elif gn_type == 'Release':
+      is_debug = False
+    else:
+      print ("Warning: could not determine build configuration from "
+             "output directory or args.gn, assuming Release build.")

[robertshield, 2016/04/05 23:43:43]

What do you think about appending some explicit wording around: "If setup.exe
fails to launch, please check your build configuration is Release."  

This should be a failure mode, so rarely encountered, and when it is
encountered, setup.exe not launching will be expected in cases where we guessed
the build mode incorrectly.

[brucedawson, 2016/04/06 00:25:05]

Done.

 
   crt_dlls = []
   sys_dll_dir = None
   if is_debug:
     crt_dlls = glob.glob(
         "C:/Program Files (x86)/Microsoft Visual Studio */VC/redist/"
         "Debug_NonRedist/" + target_arch + "/Microsoft.*.DebugCRT/*.dll")
   else:
     crt_dlls = glob.glob(
         "C:/Program Files (x86)/Microsoft Visual Studio */VC/redist/" +
@@ skipping 235 matching lines @@
                                               MINI_INSTALLER_INPUT_FILE)
 
   return options
 
 
 if '__main__' == __name__:
   options = _ParseOptions()
   if options.verbose:
     print sys.argv
   sys.exit(main(options))