dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1 | #!/usr/bin/env python |
| 2 | # Copyright 2015 The Chromium Authors. All rights reserved. |
| 3 | # Use of this source code is governed by a BSD-style license that can be |
| 4 | # found in the LICENSE file. |
| 5 | |
Dirk Pranke | 8cb6aa78 | 2017-12-16 02:31:33 | [diff] [blame] | 6 | """MB - the Meta-Build wrapper around GN. |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 7 | |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 8 | MB is a wrapper script for GN that can be used to generate build files |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 9 | for sets of canned configurations and analyze them. |
| 10 | """ |
| 11 | |
| 12 | from __future__ import print_function |
| 13 | |
| 14 | import argparse |
| 15 | import ast |
dpranke | c3441d1 | 2015-06-23 23:01:35 | [diff] [blame] | 16 | import errno |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 17 | import json |
| 18 | import os |
dpranke | 68d1cb18 | 2015-09-17 23:30:00 | [diff] [blame] | 19 | import pipes |
Dirk Pranke | 8cb6aa78 | 2017-12-16 02:31:33 | [diff] [blame] | 20 | import platform |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 21 | import pprint |
dpranke | 3cec199c | 2015-09-22 23:29:02 | [diff] [blame] | 22 | import re |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 23 | import shutil |
| 24 | import sys |
| 25 | import subprocess |
dpranke | f61de2f | 2015-05-14 04:09:56 | [diff] [blame] | 26 | import tempfile |
dpranke | bbe6d467 | 2016-04-19 06:56:57 | [diff] [blame] | 27 | import traceback |
dpranke | 867bcf4a | 2016-03-14 22:28:32 | [diff] [blame] | 28 | import urllib2 |
| 29 | |
| 30 | from collections import OrderedDict |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 31 | |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 32 | CHROMIUM_SRC_DIR = os.path.dirname(os.path.dirname(os.path.dirname( |
| 33 | os.path.abspath(__file__)))) |
| 34 | sys.path = [os.path.join(CHROMIUM_SRC_DIR, 'build')] + sys.path |
| 35 | |
| 36 | import gn_helpers |
| 37 | |
| 38 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 39 | def main(args): |
dpranke | ee5b51f6 | 2015-04-09 00:03:22 | [diff] [blame] | 40 | mbw = MetaBuildWrapper() |
dpranke | 255085e | 2016-03-16 05:23:59 | [diff] [blame] | 41 | return mbw.Main(args) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 42 | |
| 43 | |
| 44 | class MetaBuildWrapper(object): |
| 45 | def __init__(self): |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 46 | self.chromium_src_dir = CHROMIUM_SRC_DIR |
| 47 | self.default_config = os.path.join(self.chromium_src_dir, 'tools', 'mb', |
| 48 | 'mb_config.pyl') |
kjellander | 902bcb6 | 2016-10-26 06:20:50 | [diff] [blame] | 49 | self.default_isolate_map = os.path.join(self.chromium_src_dir, 'testing', |
| 50 | 'buildbot', 'gn_isolate_map.pyl') |
dpranke | 8c2cfd3 | 2015-09-17 20:12:33 | [diff] [blame] | 51 | self.executable = sys.executable |
dpranke | d1fba48 | 2015-04-14 20:54:51 | [diff] [blame] | 52 | self.platform = sys.platform |
dpranke | 8c2cfd3 | 2015-09-17 20:12:33 | [diff] [blame] | 53 | self.sep = os.sep |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 54 | self.args = argparse.Namespace() |
| 55 | self.configs = {} |
Peter Collingbourne | 4dfb64a | 2017-08-12 01:00:55 | [diff] [blame] | 56 | self.luci_tryservers = {} |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 57 | self.masters = {} |
| 58 | self.mixins = {} |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 59 | |
dpranke | 255085e | 2016-03-16 05:23:59 | [diff] [blame] | 60 | def Main(self, args): |
| 61 | self.ParseArgs(args) |
| 62 | try: |
| 63 | ret = self.args.func() |
| 64 | if ret: |
| 65 | self.DumpInputFiles() |
| 66 | return ret |
| 67 | except KeyboardInterrupt: |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 68 | self.Print('interrupted, exiting') |
dpranke | 255085e | 2016-03-16 05:23:59 | [diff] [blame] | 69 | return 130 |
dpranke | bbe6d467 | 2016-04-19 06:56:57 | [diff] [blame] | 70 | except Exception: |
dpranke | 255085e | 2016-03-16 05:23:59 | [diff] [blame] | 71 | self.DumpInputFiles() |
dpranke | bbe6d467 | 2016-04-19 06:56:57 | [diff] [blame] | 72 | s = traceback.format_exc() |
| 73 | for l in s.splitlines(): |
| 74 | self.Print(l) |
dpranke | 255085e | 2016-03-16 05:23:59 | [diff] [blame] | 75 | return 1 |
| 76 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 77 | def ParseArgs(self, argv): |
| 78 | def AddCommonOptions(subp): |
| 79 | subp.add_argument('-b', '--builder', |
| 80 | help='builder name to look up config from') |
| 81 | subp.add_argument('-m', '--master', |
| 82 | help='master name to look up config from') |
| 83 | subp.add_argument('-c', '--config', |
| 84 | help='configuration to analyze') |
shenghuazhang | 804b2154 | 2016-10-11 02:06:49 | [diff] [blame] | 85 | subp.add_argument('--phase', |
| 86 | help='optional phase name (used when builders ' |
| 87 | 'do multiple compiles with different ' |
| 88 | 'arguments in a single build)') |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 89 | subp.add_argument('-f', '--config-file', metavar='PATH', |
| 90 | default=self.default_config, |
| 91 | help='path to config file ' |
kjellander | 902bcb6 | 2016-10-26 06:20:50 | [diff] [blame] | 92 | '(default is %(default)s)') |
| 93 | subp.add_argument('-i', '--isolate-map-file', metavar='PATH', |
kjellander | 902bcb6 | 2016-10-26 06:20:50 | [diff] [blame] | 94 | help='path to isolate map file ' |
Zhiling Huang | 6695846 | 2018-02-03 00:28:20 | [diff] [blame] | 95 | '(default is %(default)s)', |
| 96 | default=[], |
| 97 | action='append', |
| 98 | dest='isolate_map_files') |
dpranke | d0c138b | 2016-04-13 18:28:47 | [diff] [blame] | 99 | subp.add_argument('-g', '--goma-dir', |
| 100 | help='path to goma directory') |
agrieve | 41d21a7 | 2016-04-14 18:02:26 | [diff] [blame] | 101 | subp.add_argument('--android-version-code', |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 102 | help='Sets GN arg android_default_version_code') |
agrieve | 41d21a7 | 2016-04-14 18:02:26 | [diff] [blame] | 103 | subp.add_argument('--android-version-name', |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 104 | help='Sets GN arg android_default_version_name') |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 105 | subp.add_argument('-n', '--dryrun', action='store_true', |
| 106 | help='Do a dry run (i.e., do nothing, just print ' |
| 107 | 'the commands that will run)') |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 108 | subp.add_argument('-v', '--verbose', action='store_true', |
| 109 | help='verbose logging') |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 110 | |
| 111 | parser = argparse.ArgumentParser(prog='mb') |
| 112 | subps = parser.add_subparsers() |
| 113 | |
| 114 | subp = subps.add_parser('analyze', |
| 115 | help='analyze whether changes to a set of files ' |
| 116 | 'will cause a set of binaries to be rebuilt.') |
| 117 | AddCommonOptions(subp) |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 118 | subp.add_argument('path', nargs=1, |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 119 | help='path build was generated into.') |
| 120 | subp.add_argument('input_path', nargs=1, |
| 121 | help='path to a file containing the input arguments ' |
| 122 | 'as a JSON object.') |
| 123 | subp.add_argument('output_path', nargs=1, |
| 124 | help='path to a file containing the output arguments ' |
| 125 | 'as a JSON object.') |
| 126 | subp.set_defaults(func=self.CmdAnalyze) |
| 127 | |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 128 | subp = subps.add_parser('export', |
| 129 | help='print out the expanded configuration for' |
| 130 | 'each builder as a JSON object') |
| 131 | subp.add_argument('-f', '--config-file', metavar='PATH', |
| 132 | default=self.default_config, |
kjellander | 902bcb6 | 2016-10-26 06:20:50 | [diff] [blame] | 133 | help='path to config file (default is %(default)s)') |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 134 | subp.add_argument('-g', '--goma-dir', |
| 135 | help='path to goma directory') |
| 136 | subp.set_defaults(func=self.CmdExport) |
| 137 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 138 | subp = subps.add_parser('gen', |
| 139 | help='generate a new set of build files') |
| 140 | AddCommonOptions(subp) |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 141 | subp.add_argument('--swarming-targets-file', |
| 142 | help='save runtime dependencies for targets listed ' |
| 143 | 'in file.') |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 144 | subp.add_argument('path', nargs=1, |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 145 | help='path to generate build into') |
| 146 | subp.set_defaults(func=self.CmdGen) |
| 147 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 148 | subp = subps.add_parser('isolate', |
| 149 | help='generate the .isolate files for a given' |
| 150 | 'binary') |
| 151 | AddCommonOptions(subp) |
| 152 | subp.add_argument('path', nargs=1, |
| 153 | help='path build was generated into') |
| 154 | subp.add_argument('target', nargs=1, |
| 155 | help='ninja target to generate the isolate for') |
| 156 | subp.set_defaults(func=self.CmdIsolate) |
| 157 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 158 | subp = subps.add_parser('lookup', |
| 159 | help='look up the command for a given config or ' |
| 160 | 'builder') |
| 161 | AddCommonOptions(subp) |
| 162 | subp.set_defaults(func=self.CmdLookup) |
| 163 | |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 164 | subp = subps.add_parser( |
| 165 | 'run', |
| 166 | help='build and run the isolated version of a ' |
| 167 | 'binary', |
| 168 | formatter_class=argparse.RawDescriptionHelpFormatter) |
| 169 | subp.description = ( |
| 170 | 'Build, isolate, and run the given binary with the command line\n' |
| 171 | 'listed in the isolate. You may pass extra arguments after the\n' |
| 172 | 'target; use "--" if the extra arguments need to include switches.\n' |
| 173 | '\n' |
| 174 | 'Examples:\n' |
| 175 | '\n' |
| 176 | ' % tools/mb/mb.py run -m chromium.linux -b "Linux Builder" \\\n' |
| 177 | ' //out/Default content_browsertests\n' |
| 178 | '\n' |
| 179 | ' % tools/mb/mb.py run out/Default content_browsertests\n' |
| 180 | '\n' |
| 181 | ' % tools/mb/mb.py run out/Default content_browsertests -- \\\n' |
| 182 | ' --test-launcher-retry-limit=0' |
| 183 | '\n' |
| 184 | ) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 185 | AddCommonOptions(subp) |
| 186 | subp.add_argument('-j', '--jobs', dest='jobs', type=int, |
| 187 | help='Number of jobs to pass to ninja') |
| 188 | subp.add_argument('--no-build', dest='build', default=True, |
| 189 | action='store_false', |
| 190 | help='Do not build, just isolate and run') |
| 191 | subp.add_argument('path', nargs=1, |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 192 | help=('path to generate build into (or use).' |
| 193 | ' This can be either a regular path or a ' |
| 194 | 'GN-style source-relative path like ' |
| 195 | '//out/Default.')) |
Dirk Pranke | 8cb6aa78 | 2017-12-16 02:31:33 | [diff] [blame] | 196 | subp.add_argument('-s', '--swarmed', action='store_true', |
| 197 | help='Run under swarming with the default dimensions') |
| 198 | subp.add_argument('-d', '--dimension', default=[], action='append', nargs=2, |
| 199 | dest='dimensions', metavar='FOO bar', |
| 200 | help='dimension to filter on') |
| 201 | subp.add_argument('--no-default-dimensions', action='store_false', |
| 202 | dest='default_dimensions', default=True, |
| 203 | help='Do not automatically add dimensions to the task') |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 204 | subp.add_argument('target', nargs=1, |
| 205 | help='ninja target to build and run') |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 206 | subp.add_argument('extra_args', nargs='*', |
| 207 | help=('extra args to pass to the isolate to run. Use ' |
| 208 | '"--" as the first arg if you need to pass ' |
| 209 | 'switches')) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 210 | subp.set_defaults(func=self.CmdRun) |
| 211 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 212 | subp = subps.add_parser('validate', |
| 213 | help='validate the config file') |
dpranke | a5a77ca | 2015-07-16 23:24:17 | [diff] [blame] | 214 | subp.add_argument('-f', '--config-file', metavar='PATH', |
| 215 | default=self.default_config, |
kjellander | 902bcb6 | 2016-10-26 06:20:50 | [diff] [blame] | 216 | help='path to config file (default is %(default)s)') |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 217 | subp.set_defaults(func=self.CmdValidate) |
| 218 | |
Peter Collingbourne | 4dfb64a | 2017-08-12 01:00:55 | [diff] [blame] | 219 | subp = subps.add_parser('gerrit-buildbucket-config', |
| 220 | help='Print buildbucket.config for gerrit ' |
| 221 | '(see MB user guide)') |
| 222 | subp.add_argument('-f', '--config-file', metavar='PATH', |
| 223 | default=self.default_config, |
| 224 | help='path to config file (default is %(default)s)') |
| 225 | subp.set_defaults(func=self.CmdBuildbucket) |
| 226 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 227 | subp = subps.add_parser('help', |
| 228 | help='Get help on a subcommand.') |
| 229 | subp.add_argument(nargs='?', action='store', dest='subcommand', |
| 230 | help='The command to get help for.') |
| 231 | subp.set_defaults(func=self.CmdHelp) |
| 232 | |
| 233 | self.args = parser.parse_args(argv) |
| 234 | |
dpranke | b2be10a | 2016-02-22 17:11:00 | [diff] [blame] | 235 | def DumpInputFiles(self): |
| 236 | |
dpranke | f7b7eb7a | 2016-03-28 22:42:59 | [diff] [blame] | 237 | def DumpContentsOfFilePassedTo(arg_name, path): |
dpranke | b2be10a | 2016-02-22 17:11:00 | [diff] [blame] | 238 | if path and self.Exists(path): |
dpranke | f7b7eb7a | 2016-03-28 22:42:59 | [diff] [blame] | 239 | self.Print("\n# To recreate the file passed to %s:" % arg_name) |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 240 | self.Print("%% cat > %s <<EOF" % path) |
dpranke | b2be10a | 2016-02-22 17:11:00 | [diff] [blame] | 241 | contents = self.ReadFile(path) |
dpranke | f7b7eb7a | 2016-03-28 22:42:59 | [diff] [blame] | 242 | self.Print(contents) |
| 243 | self.Print("EOF\n%\n") |
dpranke | b2be10a | 2016-02-22 17:11:00 | [diff] [blame] | 244 | |
dpranke | f7b7eb7a | 2016-03-28 22:42:59 | [diff] [blame] | 245 | if getattr(self.args, 'input_path', None): |
| 246 | DumpContentsOfFilePassedTo( |
| 247 | 'argv[0] (input_path)', self.args.input_path[0]) |
| 248 | if getattr(self.args, 'swarming_targets_file', None): |
| 249 | DumpContentsOfFilePassedTo( |
| 250 | '--swarming-targets-file', self.args.swarming_targets_file) |
dpranke | b2be10a | 2016-02-22 17:11:00 | [diff] [blame] | 251 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 252 | def CmdAnalyze(self): |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 253 | vals = self.Lookup() |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 254 | return self.RunGNAnalyze(vals) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 255 | |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 256 | def CmdExport(self): |
| 257 | self.ReadConfigFile() |
| 258 | obj = {} |
| 259 | for master, builders in self.masters.items(): |
| 260 | obj[master] = {} |
| 261 | for builder in builders: |
| 262 | config = self.masters[master][builder] |
| 263 | if not config: |
| 264 | continue |
| 265 | |
shenghuazhang | 804b2154 | 2016-10-11 02:06:49 | [diff] [blame] | 266 | if isinstance(config, dict): |
| 267 | args = {k: self.FlattenConfig(v)['gn_args'] |
| 268 | for k, v in config.items()} |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 269 | elif config.startswith('//'): |
| 270 | args = config |
| 271 | else: |
| 272 | args = self.FlattenConfig(config)['gn_args'] |
| 273 | if 'error' in args: |
| 274 | continue |
| 275 | |
| 276 | obj[master][builder] = args |
| 277 | |
| 278 | # Dump object and trim trailing whitespace. |
| 279 | s = '\n'.join(l.rstrip() for l in |
| 280 | json.dumps(obj, sort_keys=True, indent=2).splitlines()) |
| 281 | self.Print(s) |
| 282 | return 0 |
| 283 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 284 | def CmdGen(self): |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 285 | vals = self.Lookup() |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 286 | return self.RunGNGen(vals) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 287 | |
| 288 | def CmdHelp(self): |
| 289 | if self.args.subcommand: |
| 290 | self.ParseArgs([self.args.subcommand, '--help']) |
| 291 | else: |
| 292 | self.ParseArgs(['--help']) |
| 293 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 294 | def CmdIsolate(self): |
| 295 | vals = self.GetConfig() |
| 296 | if not vals: |
| 297 | return 1 |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 298 | return self.RunGNIsolate(vals) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 299 | |
| 300 | def CmdLookup(self): |
| 301 | vals = self.Lookup() |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 302 | cmd = self.GNCmd('gen', '_path_') |
| 303 | gn_args = self.GNArgs(vals) |
| 304 | self.Print('\nWriting """\\\n%s""" to _path_/args.gn.\n' % gn_args) |
| 305 | env = None |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 306 | |
| 307 | self.PrintCmd(cmd, env) |
| 308 | return 0 |
| 309 | |
| 310 | def CmdRun(self): |
| 311 | vals = self.GetConfig() |
| 312 | if not vals: |
| 313 | return 1 |
| 314 | |
| 315 | build_dir = self.args.path[0] |
| 316 | target = self.args.target[0] |
| 317 | |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 318 | if self.args.build: |
| 319 | ret = self.Build(target) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 320 | if ret: |
| 321 | return ret |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 322 | ret = self.RunGNIsolate(vals) |
| 323 | if ret: |
| 324 | return ret |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 325 | |
Dirk Pranke | 8cb6aa78 | 2017-12-16 02:31:33 | [diff] [blame] | 326 | if self.args.swarmed: |
| 327 | return self._RunUnderSwarming(build_dir, target) |
| 328 | else: |
| 329 | return self._RunLocallyIsolated(build_dir, target) |
| 330 | |
| 331 | def _RunUnderSwarming(self, build_dir, target): |
| 332 | # TODO(dpranke): Look up the information for the target in |
| 333 | # the //testing/buildbot.json file, if possible, so that we |
| 334 | # can determine the isolate target, command line, and additional |
| 335 | # swarming parameters, if possible. |
| 336 | # |
| 337 | # TODO(dpranke): Also, add support for sharding and merging results. |
| 338 | dimensions = [] |
| 339 | for k, v in self._DefaultDimensions() + self.args.dimensions: |
| 340 | dimensions += ['-d', k, v] |
| 341 | |
| 342 | cmd = [ |
| 343 | self.executable, |
| 344 | self.PathJoin('tools', 'swarming_client', 'isolate.py'), |
| 345 | 'archive', |
| 346 | '-s', |
| 347 | self.ToSrcRelPath('%s/%s.isolated' % (build_dir, target)), |
| 348 | '-I', 'isolateserver.appspot.com', |
| 349 | ] |
| 350 | ret, out, _ = self.Run(cmd, force_verbose=False) |
| 351 | if ret: |
| 352 | return ret |
| 353 | |
| 354 | isolated_hash = out.splitlines()[0].split()[0] |
| 355 | cmd = [ |
| 356 | self.executable, |
| 357 | self.PathJoin('tools', 'swarming_client', 'swarming.py'), |
| 358 | 'run', |
| 359 | '-s', isolated_hash, |
| 360 | '-I', 'isolateserver.appspot.com', |
| 361 | '-S', 'chromium-swarm.appspot.com', |
| 362 | ] + dimensions |
| 363 | if self.args.extra_args: |
| 364 | cmd += ['--'] + self.args.extra_args |
| 365 | ret, _, _ = self.Run(cmd, force_verbose=True, buffer_output=False) |
| 366 | return ret |
| 367 | |
| 368 | def _RunLocallyIsolated(self, build_dir, target): |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 369 | cmd = [ |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 370 | self.executable, |
| 371 | self.PathJoin('tools', 'swarming_client', 'isolate.py'), |
| 372 | 'run', |
| 373 | '-s', |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 374 | self.ToSrcRelPath('%s/%s.isolated' % (build_dir, target)), |
Dirk Pranke | 8cb6aa78 | 2017-12-16 02:31:33 | [diff] [blame] | 375 | ] |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 376 | if self.args.extra_args: |
Dirk Pranke | 8cb6aa78 | 2017-12-16 02:31:33 | [diff] [blame] | 377 | cmd += ['--'] + self.args.extra_args |
| 378 | ret, _, _ = self.Run(cmd, force_verbose=True, buffer_output=False) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 379 | return ret |
| 380 | |
Dirk Pranke | 8cb6aa78 | 2017-12-16 02:31:33 | [diff] [blame] | 381 | def _DefaultDimensions(self): |
| 382 | if not self.args.default_dimensions: |
| 383 | return [] |
| 384 | |
| 385 | # This code is naive and just picks reasonable defaults per platform. |
Nico Weber | d94b71a | 2018-02-22 22:00:30 | [diff] [blame^] | 386 | # TODO(thakis): This assumes that host platform is the same as |
| 387 | # target platform. |
Dirk Pranke | 8cb6aa78 | 2017-12-16 02:31:33 | [diff] [blame] | 388 | if self.platform == 'darwin': |
| 389 | os_dim = ('os', 'Mac-10.12') |
| 390 | elif self.platform.startswith('linux'): |
| 391 | os_dim = ('os', 'Ubuntu-14.04') |
| 392 | elif self.platform == 'win32': |
| 393 | os_dim = ('os', 'Windows-10-14393') |
| 394 | else: |
| 395 | raise MBErr('unrecognized platform string "%s"' % self.platform) |
| 396 | |
| 397 | return [('pool', 'Chrome'), |
| 398 | ('cpu', 'x86-64'), |
| 399 | os_dim] |
| 400 | |
Peter Collingbourne | 4dfb64a | 2017-08-12 01:00:55 | [diff] [blame] | 401 | def CmdBuildbucket(self): |
| 402 | self.ReadConfigFile() |
| 403 | |
| 404 | self.Print('# This file was generated using ' |
| 405 | '"tools/mb/mb.py gerrit-buildbucket-config".') |
| 406 | |
| 407 | for luci_tryserver in sorted(self.luci_tryservers): |
| 408 | self.Print('[bucket "luci.%s"]' % luci_tryserver) |
| 409 | for bot in sorted(self.luci_tryservers[luci_tryserver]): |
| 410 | self.Print('\tbuilder = %s' % bot) |
| 411 | |
| 412 | for master in sorted(self.masters): |
| 413 | if master.startswith('tryserver.'): |
| 414 | self.Print('[bucket "master.%s"]' % master) |
| 415 | for bot in sorted(self.masters[master]): |
| 416 | self.Print('\tbuilder = %s' % bot) |
| 417 | |
| 418 | return 0 |
| 419 | |
dpranke | 0cafc16 | 2016-03-19 00:41:10 | [diff] [blame] | 420 | def CmdValidate(self, print_ok=True): |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 421 | errs = [] |
| 422 | |
| 423 | # Read the file to make sure it parses. |
| 424 | self.ReadConfigFile() |
| 425 | |
dpranke | 3be0014 | 2016-03-17 22:46:04 | [diff] [blame] | 426 | # Build a list of all of the configs referenced by builders. |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 427 | all_configs = {} |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 428 | for master in self.masters: |
dpranke | 3be0014 | 2016-03-17 22:46:04 | [diff] [blame] | 429 | for config in self.masters[master].values(): |
shenghuazhang | 804b2154 | 2016-10-11 02:06:49 | [diff] [blame] | 430 | if isinstance(config, dict): |
| 431 | for c in config.values(): |
dpranke | b9380a1 | 2016-07-21 21:44:09 | [diff] [blame] | 432 | all_configs[c] = master |
| 433 | else: |
| 434 | all_configs[config] = master |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 435 | |
dpranke | 9dd5e25 | 2016-04-14 04:23:09 | [diff] [blame] | 436 | # Check that every referenced args file or config actually exists. |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 437 | for config, loc in all_configs.items(): |
dpranke | 9dd5e25 | 2016-04-14 04:23:09 | [diff] [blame] | 438 | if config.startswith('//'): |
| 439 | if not self.Exists(self.ToAbsPath(config)): |
| 440 | errs.append('Unknown args file "%s" referenced from "%s".' % |
| 441 | (config, loc)) |
| 442 | elif not config in self.configs: |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 443 | errs.append('Unknown config "%s" referenced from "%s".' % |
| 444 | (config, loc)) |
| 445 | |
| 446 | # Check that every actual config is actually referenced. |
| 447 | for config in self.configs: |
| 448 | if not config in all_configs: |
| 449 | errs.append('Unused config "%s".' % config) |
| 450 | |
| 451 | # Figure out the whole list of mixins, and check that every mixin |
| 452 | # listed by a config or another mixin actually exists. |
| 453 | referenced_mixins = set() |
| 454 | for config, mixins in self.configs.items(): |
| 455 | for mixin in mixins: |
| 456 | if not mixin in self.mixins: |
| 457 | errs.append('Unknown mixin "%s" referenced by config "%s".' % |
| 458 | (mixin, config)) |
| 459 | referenced_mixins.add(mixin) |
| 460 | |
| 461 | for mixin in self.mixins: |
| 462 | for sub_mixin in self.mixins[mixin].get('mixins', []): |
| 463 | if not sub_mixin in self.mixins: |
| 464 | errs.append('Unknown mixin "%s" referenced by mixin "%s".' % |
| 465 | (sub_mixin, mixin)) |
| 466 | referenced_mixins.add(sub_mixin) |
| 467 | |
| 468 | # Check that every mixin defined is actually referenced somewhere. |
| 469 | for mixin in self.mixins: |
| 470 | if not mixin in referenced_mixins: |
| 471 | errs.append('Unreferenced mixin "%s".' % mixin) |
| 472 | |
dpranke | 255085e | 2016-03-16 05:23:59 | [diff] [blame] | 473 | # If we're checking the Chromium config, check that the 'chromium' bots |
| 474 | # which build public artifacts do not include the chrome_with_codecs mixin. |
| 475 | if self.args.config_file == self.default_config: |
| 476 | if 'chromium' in self.masters: |
| 477 | for builder in self.masters['chromium']: |
| 478 | config = self.masters['chromium'][builder] |
| 479 | def RecurseMixins(current_mixin): |
| 480 | if current_mixin == 'chrome_with_codecs': |
| 481 | errs.append('Public artifact builder "%s" can not contain the ' |
| 482 | '"chrome_with_codecs" mixin.' % builder) |
| 483 | return |
| 484 | if not 'mixins' in self.mixins[current_mixin]: |
| 485 | return |
| 486 | for mixin in self.mixins[current_mixin]['mixins']: |
| 487 | RecurseMixins(mixin) |
dalecurtis | 56fd27e | 2016-03-09 23:06:41 | [diff] [blame] | 488 | |
dpranke | 255085e | 2016-03-16 05:23:59 | [diff] [blame] | 489 | for mixin in self.configs[config]: |
| 490 | RecurseMixins(mixin) |
| 491 | else: |
| 492 | errs.append('Missing "chromium" master. Please update this ' |
| 493 | 'proprietary codecs check with the name of the master ' |
| 494 | 'responsible for public build artifacts.') |
dalecurtis | 56fd27e | 2016-03-09 23:06:41 | [diff] [blame] | 495 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 496 | if errs: |
dpranke | 4323c8063 | 2015-08-10 22:53:54 | [diff] [blame] | 497 | raise MBErr(('mb config file %s has problems:' % self.args.config_file) + |
dpranke | a3326787 | 2015-08-12 15:45:17 | [diff] [blame] | 498 | '\n ' + '\n '.join(errs)) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 499 | |
dpranke | 0cafc16 | 2016-03-19 00:41:10 | [diff] [blame] | 500 | if print_ok: |
| 501 | self.Print('mb config file %s looks ok.' % self.args.config_file) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 502 | return 0 |
| 503 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 504 | def GetConfig(self): |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 505 | build_dir = self.args.path[0] |
| 506 | |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 507 | vals = self.DefaultVals() |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 508 | if self.args.builder or self.args.master or self.args.config: |
| 509 | vals = self.Lookup() |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 510 | # Re-run gn gen in order to ensure the config is consistent with the |
| 511 | # build dir. |
| 512 | self.RunGNGen(vals) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 513 | return vals |
| 514 | |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 515 | toolchain_path = self.PathJoin(self.ToAbsPath(build_dir), |
| 516 | 'toolchain.ninja') |
| 517 | if not self.Exists(toolchain_path): |
| 518 | self.Print('Must either specify a path to an existing GN build dir ' |
| 519 | 'or pass in a -m/-b pair or a -c flag to specify the ' |
| 520 | 'configuration') |
| 521 | return {} |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 522 | |
Dirk Pranke | d181a1a | 2017-12-14 01:47:11 | [diff] [blame] | 523 | vals['gn_args'] = self.GNArgsFromDir(build_dir) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 524 | return vals |
| 525 | |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 526 | def GNArgsFromDir(self, build_dir): |
brucedawson | ecc0c1cd | 2016-06-02 18:24:58 | [diff] [blame] | 527 | args_contents = "" |
| 528 | gn_args_path = self.PathJoin(self.ToAbsPath(build_dir), 'args.gn') |
| 529 | if self.Exists(gn_args_path): |
| 530 | args_contents = self.ReadFile(gn_args_path) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 531 | gn_args = [] |
| 532 | for l in args_contents.splitlines(): |
| 533 | fields = l.split(' ') |
| 534 | name = fields[0] |
| 535 | val = ' '.join(fields[2:]) |
| 536 | gn_args.append('%s=%s' % (name, val)) |
| 537 | |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 538 | return ' '.join(gn_args) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 539 | |
| 540 | def Lookup(self): |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 541 | vals = self.ReadIOSBotConfig() |
dpranke | e0f486f | 2015-11-19 23:42:00 | [diff] [blame] | 542 | if not vals: |
| 543 | self.ReadConfigFile() |
| 544 | config = self.ConfigFromArgs() |
dpranke | 9dd5e25 | 2016-04-14 04:23:09 | [diff] [blame] | 545 | if config.startswith('//'): |
| 546 | if not self.Exists(self.ToAbsPath(config)): |
| 547 | raise MBErr('args file "%s" not found' % config) |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 548 | vals = self.DefaultVals() |
| 549 | vals['args_file'] = config |
dpranke | 9dd5e25 | 2016-04-14 04:23:09 | [diff] [blame] | 550 | else: |
| 551 | if not config in self.configs: |
| 552 | raise MBErr('Config "%s" not found in %s' % |
| 553 | (config, self.args.config_file)) |
| 554 | vals = self.FlattenConfig(config) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 555 | return vals |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 556 | |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 557 | def ReadIOSBotConfig(self): |
dpranke | e0f486f | 2015-11-19 23:42:00 | [diff] [blame] | 558 | if not self.args.master or not self.args.builder: |
| 559 | return {} |
| 560 | path = self.PathJoin(self.chromium_src_dir, 'ios', 'build', 'bots', |
| 561 | self.args.master, self.args.builder + '.json') |
| 562 | if not self.Exists(path): |
| 563 | return {} |
| 564 | |
| 565 | contents = json.loads(self.ReadFile(path)) |
dpranke | e0f486f | 2015-11-19 23:42:00 | [diff] [blame] | 566 | gn_args = ' '.join(contents.get('gn_args', [])) |
| 567 | |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 568 | vals = self.DefaultVals() |
| 569 | vals['gn_args'] = gn_args |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 570 | return vals |
dpranke | e0f486f | 2015-11-19 23:42:00 | [diff] [blame] | 571 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 572 | def ReadConfigFile(self): |
| 573 | if not self.Exists(self.args.config_file): |
| 574 | raise MBErr('config file not found at %s' % self.args.config_file) |
| 575 | |
| 576 | try: |
| 577 | contents = ast.literal_eval(self.ReadFile(self.args.config_file)) |
| 578 | except SyntaxError as e: |
| 579 | raise MBErr('Failed to parse config file "%s": %s' % |
| 580 | (self.args.config_file, e)) |
| 581 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 582 | self.configs = contents['configs'] |
Peter Collingbourne | 4dfb64a | 2017-08-12 01:00:55 | [diff] [blame] | 583 | self.luci_tryservers = contents.get('luci_tryservers', {}) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 584 | self.masters = contents['masters'] |
| 585 | self.mixins = contents['mixins'] |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 586 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 587 | def ReadIsolateMap(self): |
Zhiling Huang | 6695846 | 2018-02-03 00:28:20 | [diff] [blame] | 588 | if not self.args.isolate_map_files: |
| 589 | self.args.isolate_map_files = [self.default_isolate_map] |
| 590 | |
| 591 | for f in self.args.isolate_map_files: |
| 592 | if not self.Exists(f): |
| 593 | raise MBErr('isolate map file not found at %s' % f) |
| 594 | isolate_maps = {} |
| 595 | for isolate_map in self.args.isolate_map_files: |
| 596 | try: |
| 597 | isolate_map = ast.literal_eval(self.ReadFile(isolate_map)) |
| 598 | duplicates = set(isolate_map).intersection(isolate_maps) |
| 599 | if duplicates: |
| 600 | raise MBErr( |
| 601 | 'Duplicate targets in isolate map files: %s.' % |
| 602 | ', '.join(duplicates)) |
| 603 | isolate_maps.update(isolate_map) |
| 604 | except SyntaxError as e: |
| 605 | raise MBErr( |
| 606 | 'Failed to parse isolate map file "%s": %s' % (isolate_map, e)) |
| 607 | return isolate_maps |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 608 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 609 | def ConfigFromArgs(self): |
| 610 | if self.args.config: |
| 611 | if self.args.master or self.args.builder: |
| 612 | raise MBErr('Can not specific both -c/--config and -m/--master or ' |
| 613 | '-b/--builder') |
| 614 | |
| 615 | return self.args.config |
| 616 | |
| 617 | if not self.args.master or not self.args.builder: |
| 618 | raise MBErr('Must specify either -c/--config or ' |
| 619 | '(-m/--master and -b/--builder)') |
| 620 | |
| 621 | if not self.args.master in self.masters: |
| 622 | raise MBErr('Master name "%s" not found in "%s"' % |
| 623 | (self.args.master, self.args.config_file)) |
| 624 | |
| 625 | if not self.args.builder in self.masters[self.args.master]: |
| 626 | raise MBErr('Builder name "%s" not found under masters[%s] in "%s"' % |
| 627 | (self.args.builder, self.args.master, self.args.config_file)) |
| 628 | |
dpranke | b9380a1 | 2016-07-21 21:44:09 | [diff] [blame] | 629 | config = self.masters[self.args.master][self.args.builder] |
shenghuazhang | 804b2154 | 2016-10-11 02:06:49 | [diff] [blame] | 630 | if isinstance(config, dict): |
dpranke | b9380a1 | 2016-07-21 21:44:09 | [diff] [blame] | 631 | if self.args.phase is None: |
| 632 | raise MBErr('Must specify a build --phase for %s on %s' % |
| 633 | (self.args.builder, self.args.master)) |
shenghuazhang | 804b2154 | 2016-10-11 02:06:49 | [diff] [blame] | 634 | phase = str(self.args.phase) |
| 635 | if phase not in config: |
| 636 | raise MBErr('Phase %s doesn\'t exist for %s on %s' % |
dpranke | b9380a1 | 2016-07-21 21:44:09 | [diff] [blame] | 637 | (phase, self.args.builder, self.args.master)) |
shenghuazhang | 804b2154 | 2016-10-11 02:06:49 | [diff] [blame] | 638 | return config[phase] |
dpranke | b9380a1 | 2016-07-21 21:44:09 | [diff] [blame] | 639 | |
| 640 | if self.args.phase is not None: |
| 641 | raise MBErr('Must not specify a build --phase for %s on %s' % |
| 642 | (self.args.builder, self.args.master)) |
| 643 | return config |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 644 | |
| 645 | def FlattenConfig(self, config): |
| 646 | mixins = self.configs[config] |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 647 | vals = self.DefaultVals() |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 648 | |
| 649 | visited = [] |
| 650 | self.FlattenMixins(mixins, vals, visited) |
| 651 | return vals |
| 652 | |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 653 | def DefaultVals(self): |
| 654 | return { |
| 655 | 'args_file': '', |
| 656 | 'cros_passthrough': False, |
| 657 | 'gn_args': '', |
dpranke | f37aebb9 | 2016-09-23 01:14:49 | [diff] [blame] | 658 | } |
| 659 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 660 | def FlattenMixins(self, mixins, vals, visited): |
| 661 | for m in mixins: |
| 662 | if m not in self.mixins: |
| 663 | raise MBErr('Unknown mixin "%s"' % m) |
dpranke | ee5b51f6 | 2015-04-09 00:03:22 | [diff] [blame] | 664 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 665 | visited.append(m) |
| 666 | |
| 667 | mixin_vals = self.mixins[m] |
dpranke | 73ed0d6 | 2016-04-25 19:18:34 | [diff] [blame] | 668 | |
| 669 | if 'cros_passthrough' in mixin_vals: |
| 670 | vals['cros_passthrough'] = mixin_vals['cros_passthrough'] |
Dirk Pranke | 6b99f07 | 2017-04-05 00:58:30 | [diff] [blame] | 671 | if 'args_file' in mixin_vals: |
| 672 | if vals['args_file']: |
| 673 | raise MBErr('args_file specified multiple times in mixins ' |
| 674 | 'for %s on %s' % (self.args.builder, self.args.master)) |
| 675 | vals['args_file'] = mixin_vals['args_file'] |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 676 | if 'gn_args' in mixin_vals: |
| 677 | if vals['gn_args']: |
| 678 | vals['gn_args'] += ' ' + mixin_vals['gn_args'] |
| 679 | else: |
| 680 | vals['gn_args'] = mixin_vals['gn_args'] |
dpranke | 73ed0d6 | 2016-04-25 19:18:34 | [diff] [blame] | 681 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 682 | if 'mixins' in mixin_vals: |
| 683 | self.FlattenMixins(mixin_vals['mixins'], vals, visited) |
| 684 | return vals |
| 685 | |
Dirk Pranke | a3727f9 | 2017-07-17 17:30:33 | [diff] [blame] | 686 | def RunGNGen(self, vals, compute_grit_inputs_for_analyze=False): |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 687 | build_dir = self.args.path[0] |
Dirk Pranke | 0fd41bcd | 2015-06-19 00:05:50 | [diff] [blame] | 688 | |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 689 | cmd = self.GNCmd('gen', build_dir, '--check') |
| 690 | gn_args = self.GNArgs(vals) |
Dirk Pranke | a3727f9 | 2017-07-17 17:30:33 | [diff] [blame] | 691 | if compute_grit_inputs_for_analyze: |
| 692 | gn_args += ' compute_grit_inputs_for_analyze=true' |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 693 | |
| 694 | # Since GN hasn't run yet, the build directory may not even exist. |
| 695 | self.MaybeMakeDirectory(self.ToAbsPath(build_dir)) |
| 696 | |
| 697 | gn_args_path = self.ToAbsPath(build_dir, 'args.gn') |
dpranke | 4ff8b9f | 2016-04-15 03:07:54 | [diff] [blame] | 698 | self.WriteFile(gn_args_path, gn_args, force_verbose=True) |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 699 | |
| 700 | swarming_targets = [] |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 701 | if getattr(self.args, 'swarming_targets_file', None): |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 702 | # We need GN to generate the list of runtime dependencies for |
| 703 | # the compile targets listed (one per line) in the file so |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 704 | # we can run them via swarming. We use gn_isolate_map.pyl to convert |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 705 | # the compile targets to the matching GN labels. |
dpranke | b2be10a | 2016-02-22 17:11:00 | [diff] [blame] | 706 | path = self.args.swarming_targets_file |
| 707 | if not self.Exists(path): |
| 708 | self.WriteFailureAndRaise('"%s" does not exist' % path, |
| 709 | output_path=None) |
| 710 | contents = self.ReadFile(path) |
| 711 | swarming_targets = set(contents.splitlines()) |
dpranke | b2be10a | 2016-02-22 17:11:00 | [diff] [blame] | 712 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 713 | isolate_map = self.ReadIsolateMap() |
| 714 | err, labels = self.MapTargetsToLabels(isolate_map, swarming_targets) |
dpranke | b2be10a | 2016-02-22 17:11:00 | [diff] [blame] | 715 | if err: |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 716 | raise MBErr(err) |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 717 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 718 | gn_runtime_deps_path = self.ToAbsPath(build_dir, 'runtime_deps') |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 719 | self.WriteFile(gn_runtime_deps_path, '\n'.join(labels) + '\n') |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 720 | cmd.append('--runtime-deps-list-file=%s' % gn_runtime_deps_path) |
| 721 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 722 | ret, _, _ = self.Run(cmd) |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 723 | if ret: |
| 724 | # If `gn gen` failed, we should exit early rather than trying to |
| 725 | # generate isolates. Run() will have already logged any error output. |
| 726 | self.Print('GN gen failed: %d' % ret) |
| 727 | return ret |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 728 | |
jbudorick | e3c4f95e | 2016-04-28 23:17:38 | [diff] [blame] | 729 | android = 'target_os="android"' in vals['gn_args'] |
Kevin Marshall | f35fa5f | 2018-01-29 19:24:42 | [diff] [blame] | 730 | fuchsia = 'target_os="fuchsia"' in vals['gn_args'] |
Nico Weber | d94b71a | 2018-02-22 22:00:30 | [diff] [blame^] | 731 | win = self.platform == 'win32' or 'target_os="win"' in vals['gn_args'] |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 732 | for target in swarming_targets: |
jbudorick | e3c4f95e | 2016-04-28 23:17:38 | [diff] [blame] | 733 | if android: |
| 734 | # Android targets may be either android_apk or executable. The former |
jbudorick | 91c8a601 | 2016-01-29 23:20:02 | [diff] [blame] | 735 | # will result in runtime_deps associated with the stamp file, while the |
| 736 | # latter will result in runtime_deps associated with the executable. |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 737 | label = isolate_map[target]['label'] |
jbudorick | 91c8a601 | 2016-01-29 23:20:02 | [diff] [blame] | 738 | runtime_deps_targets = [ |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 739 | target + '.runtime_deps', |
dpranke | 48ccf8f | 2016-03-28 23:58:28 | [diff] [blame] | 740 | 'obj/%s.stamp.runtime_deps' % label.replace(':', '/')] |
Kevin Marshall | f35fa5f | 2018-01-29 19:24:42 | [diff] [blame] | 741 | elif fuchsia: |
| 742 | # Only emit a runtime deps file for the group() target on Fuchsia. |
| 743 | label = isolate_map[target]['label'] |
| 744 | runtime_deps_targets = [ |
| 745 | 'obj/%s.stamp.runtime_deps' % label.replace(':', '/')] |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 746 | elif (isolate_map[target]['type'] == 'script' or |
| 747 | isolate_map[target].get('label_type') == 'group'): |
dpranke | 6abd865 | 2015-08-28 03:21:11 | [diff] [blame] | 748 | # For script targets, the build target is usually a group, |
| 749 | # for which gn generates the runtime_deps next to the stamp file |
eyaich | 82d5ac94 | 2016-11-03 12:13:49 | [diff] [blame] | 750 | # for the label, which lives under the obj/ directory, but it may |
| 751 | # also be an executable. |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 752 | label = isolate_map[target]['label'] |
dpranke | 48ccf8f | 2016-03-28 23:58:28 | [diff] [blame] | 753 | runtime_deps_targets = [ |
| 754 | 'obj/%s.stamp.runtime_deps' % label.replace(':', '/')] |
Nico Weber | d94b71a | 2018-02-22 22:00:30 | [diff] [blame^] | 755 | if win: |
eyaich | 82d5ac94 | 2016-11-03 12:13:49 | [diff] [blame] | 756 | runtime_deps_targets += [ target + '.exe.runtime_deps' ] |
| 757 | else: |
| 758 | runtime_deps_targets += [ target + '.runtime_deps' ] |
Nico Weber | d94b71a | 2018-02-22 22:00:30 | [diff] [blame^] | 759 | elif win: |
dpranke | 48ccf8f | 2016-03-28 23:58:28 | [diff] [blame] | 760 | runtime_deps_targets = [target + '.exe.runtime_deps'] |
dpranke | 34bd39d | 2015-06-24 02:36:52 | [diff] [blame] | 761 | else: |
dpranke | 48ccf8f | 2016-03-28 23:58:28 | [diff] [blame] | 762 | runtime_deps_targets = [target + '.runtime_deps'] |
jbudorick | 91c8a601 | 2016-01-29 23:20:02 | [diff] [blame] | 763 | |
dpranke | 48ccf8f | 2016-03-28 23:58:28 | [diff] [blame] | 764 | for r in runtime_deps_targets: |
| 765 | runtime_deps_path = self.ToAbsPath(build_dir, r) |
| 766 | if self.Exists(runtime_deps_path): |
jbudorick | 91c8a601 | 2016-01-29 23:20:02 | [diff] [blame] | 767 | break |
| 768 | else: |
dpranke | 48ccf8f | 2016-03-28 23:58:28 | [diff] [blame] | 769 | raise MBErr('did not generate any of %s' % |
| 770 | ', '.join(runtime_deps_targets)) |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 771 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 772 | command, extra_files = self.GetIsolateCommand(target, vals) |
dpranke | d5b2b943 | 2015-06-23 16:55:30 | [diff] [blame] | 773 | |
dpranke | 48ccf8f | 2016-03-28 23:58:28 | [diff] [blame] | 774 | runtime_deps = self.ReadFile(runtime_deps_path).splitlines() |
dpranke | d5b2b943 | 2015-06-23 16:55:30 | [diff] [blame] | 775 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 776 | self.WriteIsolateFiles(build_dir, command, target, runtime_deps, |
| 777 | extra_files) |
dpranke | d5b2b943 | 2015-06-23 16:55:30 | [diff] [blame] | 778 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 779 | return 0 |
| 780 | |
| 781 | def RunGNIsolate(self, vals): |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 782 | target = self.args.target[0] |
| 783 | isolate_map = self.ReadIsolateMap() |
| 784 | err, labels = self.MapTargetsToLabels(isolate_map, [target]) |
| 785 | if err: |
| 786 | raise MBErr(err) |
| 787 | label = labels[0] |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 788 | |
| 789 | build_dir = self.args.path[0] |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 790 | command, extra_files = self.GetIsolateCommand(target, vals) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 791 | |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 792 | cmd = self.GNCmd('desc', build_dir, label, 'runtime_deps') |
dpranke | 40da020 | 2016-02-13 05:05:20 | [diff] [blame] | 793 | ret, out, _ = self.Call(cmd) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 794 | if ret: |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 795 | if out: |
| 796 | self.Print(out) |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 797 | return ret |
| 798 | |
| 799 | runtime_deps = out.splitlines() |
| 800 | |
| 801 | self.WriteIsolateFiles(build_dir, command, target, runtime_deps, |
| 802 | extra_files) |
| 803 | |
| 804 | ret, _, _ = self.Run([ |
| 805 | self.executable, |
| 806 | self.PathJoin('tools', 'swarming_client', 'isolate.py'), |
| 807 | 'check', |
| 808 | '-i', |
| 809 | self.ToSrcRelPath('%s/%s.isolate' % (build_dir, target)), |
| 810 | '-s', |
| 811 | self.ToSrcRelPath('%s/%s.isolated' % (build_dir, target))], |
| 812 | buffer_output=False) |
dpranke | d5b2b943 | 2015-06-23 16:55:30 | [diff] [blame] | 813 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 814 | return ret |
| 815 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 816 | def WriteIsolateFiles(self, build_dir, command, target, runtime_deps, |
| 817 | extra_files): |
| 818 | isolate_path = self.ToAbsPath(build_dir, target + '.isolate') |
| 819 | self.WriteFile(isolate_path, |
| 820 | pprint.pformat({ |
| 821 | 'variables': { |
| 822 | 'command': command, |
| 823 | 'files': sorted(runtime_deps + extra_files), |
| 824 | } |
| 825 | }) + '\n') |
| 826 | |
| 827 | self.WriteJSON( |
| 828 | { |
| 829 | 'args': [ |
| 830 | '--isolated', |
| 831 | self.ToSrcRelPath('%s/%s.isolated' % (build_dir, target)), |
| 832 | '--isolate', |
| 833 | self.ToSrcRelPath('%s/%s.isolate' % (build_dir, target)), |
| 834 | ], |
| 835 | 'dir': self.chromium_src_dir, |
| 836 | 'version': 1, |
| 837 | }, |
| 838 | isolate_path + 'd.gen.json', |
| 839 | ) |
| 840 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 841 | def MapTargetsToLabels(self, isolate_map, targets): |
| 842 | labels = [] |
| 843 | err = '' |
| 844 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 845 | for target in targets: |
| 846 | if target == 'all': |
| 847 | labels.append(target) |
| 848 | elif target.startswith('//'): |
| 849 | labels.append(target) |
| 850 | else: |
| 851 | if target in isolate_map: |
thakis | 024d6f3 | 2017-05-16 23:21:42 | [diff] [blame] | 852 | if isolate_map[target]['type'] == 'unknown': |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 853 | err += ('test target "%s" type is unknown\n' % target) |
| 854 | else: |
thakis | 024d6f3 | 2017-05-16 23:21:42 | [diff] [blame] | 855 | labels.append(isolate_map[target]['label']) |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 856 | else: |
| 857 | err += ('target "%s" not found in ' |
| 858 | '//testing/buildbot/gn_isolate_map.pyl\n' % target) |
| 859 | |
| 860 | return err, labels |
| 861 | |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 862 | def GNCmd(self, subcommand, path, *args): |
dpranke | d1fba48 | 2015-04-14 20:54:51 | [diff] [blame] | 863 | if self.platform == 'linux2': |
dpranke | 40da020 | 2016-02-13 05:05:20 | [diff] [blame] | 864 | subdir, exe = 'linux64', 'gn' |
dpranke | d1fba48 | 2015-04-14 20:54:51 | [diff] [blame] | 865 | elif self.platform == 'darwin': |
dpranke | 40da020 | 2016-02-13 05:05:20 | [diff] [blame] | 866 | subdir, exe = 'mac', 'gn' |
dpranke | d1fba48 | 2015-04-14 20:54:51 | [diff] [blame] | 867 | else: |
dpranke | 40da020 | 2016-02-13 05:05:20 | [diff] [blame] | 868 | subdir, exe = 'win', 'gn.exe' |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 869 | |
dpranke | 40da020 | 2016-02-13 05:05:20 | [diff] [blame] | 870 | gn_path = self.PathJoin(self.chromium_src_dir, 'buildtools', subdir, exe) |
dpranke | 10118bf | 2016-09-16 23:16:08 | [diff] [blame] | 871 | return [gn_path, subcommand, path] + list(args) |
dpranke | 9aba8b21 | 2016-09-16 22:52:52 | [diff] [blame] | 872 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 873 | |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 874 | def GNArgs(self, vals): |
dpranke | 73ed0d6 | 2016-04-25 19:18:34 | [diff] [blame] | 875 | if vals['cros_passthrough']: |
| 876 | if not 'GN_ARGS' in os.environ: |
| 877 | raise MBErr('MB is expecting GN_ARGS to be in the environment') |
| 878 | gn_args = os.environ['GN_ARGS'] |
dpranke | 4026018 | 2016-04-27 04:45:16 | [diff] [blame] | 879 | if not re.search('target_os.*=.*"chromeos"', gn_args): |
dpranke | 39f3be0 | 2016-04-27 04:07:30 | [diff] [blame] | 880 | raise MBErr('GN_ARGS is missing target_os = "chromeos": (GN_ARGS=%s)' % |
dpranke | 73ed0d6 | 2016-04-25 19:18:34 | [diff] [blame] | 881 | gn_args) |
| 882 | else: |
| 883 | gn_args = vals['gn_args'] |
| 884 | |
dpranke | d0c138b | 2016-04-13 18:28:47 | [diff] [blame] | 885 | if self.args.goma_dir: |
| 886 | gn_args += ' goma_dir="%s"' % self.args.goma_dir |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 887 | |
agrieve | 41d21a7 | 2016-04-14 18:02:26 | [diff] [blame] | 888 | android_version_code = self.args.android_version_code |
| 889 | if android_version_code: |
| 890 | gn_args += ' android_default_version_code="%s"' % android_version_code |
| 891 | |
| 892 | android_version_name = self.args.android_version_name |
| 893 | if android_version_name: |
| 894 | gn_args += ' android_default_version_name="%s"' % android_version_name |
| 895 | |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 896 | # Canonicalize the arg string into a sorted, newline-separated list |
| 897 | # of key-value pairs, and de-dup the keys if need be so that only |
| 898 | # the last instance of each arg is listed. |
| 899 | gn_args = gn_helpers.ToGNString(gn_helpers.FromGNArgs(gn_args)) |
| 900 | |
dpranke | 9dd5e25 | 2016-04-14 04:23:09 | [diff] [blame] | 901 | args_file = vals.get('args_file', None) |
| 902 | if args_file: |
| 903 | gn_args = ('import("%s")\n' % vals['args_file']) + gn_args |
dpranke | eca4a78 | 2016-04-14 01:42:38 | [diff] [blame] | 904 | return gn_args |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 905 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 906 | def GetIsolateCommand(self, target, vals): |
kylechar | 50abf5a | 2016-11-29 16:03:07 | [diff] [blame] | 907 | isolate_map = self.ReadIsolateMap() |
| 908 | |
Scott Graham | 3be4b416 | 2017-09-12 00:41:41 | [diff] [blame] | 909 | is_android = 'target_os="android"' in vals['gn_args'] |
| 910 | is_fuchsia = 'target_os="fuchsia"' in vals['gn_args'] |
Nico Weber | d94b71a | 2018-02-22 22:00:30 | [diff] [blame^] | 911 | is_win = self.platform == 'win32' or 'target_os="win"' in vals['gn_args'] |
jbudorick | e842873 | 2016-02-02 02:17:06 | [diff] [blame] | 912 | |
kylechar | 3970568 | 2017-01-19 14:37:23 | [diff] [blame] | 913 | # This should be true if tests with type='windowed_test_launcher' are |
| 914 | # expected to run using xvfb. For example, Linux Desktop, X11 CrOS and |
msisov | aea5273 | 2017-03-21 08:08:08 | [diff] [blame] | 915 | # Ozone CrOS builds. Note that one Ozone build can be used to run differen |
| 916 | # backends. Currently, tests are executed for the headless and X11 backends |
| 917 | # and both can run under Xvfb. |
| 918 | # TODO(tonikitoo,msisov,fwang): Find a way to run tests for the Wayland |
| 919 | # backend. |
Scott Graham | 3be4b416 | 2017-09-12 00:41:41 | [diff] [blame] | 920 | use_xvfb = self.platform == 'linux2' and not is_android and not is_fuchsia |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 921 | |
| 922 | asan = 'is_asan=true' in vals['gn_args'] |
| 923 | msan = 'is_msan=true' in vals['gn_args'] |
| 924 | tsan = 'is_tsan=true' in vals['gn_args'] |
pcc | 46233c2 | 2017-06-20 22:11:41 | [diff] [blame] | 925 | cfi_diag = 'use_cfi_diag=true' in vals['gn_args'] |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 926 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 927 | test_type = isolate_map[target]['type'] |
dpranke | fe0d35e | 2016-02-05 02:43:59 | [diff] [blame] | 928 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 929 | executable = isolate_map[target].get('executable', target) |
Nico Weber | d94b71a | 2018-02-22 22:00:30 | [diff] [blame^] | 930 | executable_suffix = '.exe' if is_win else '' |
dpranke | fe0d35e | 2016-02-05 02:43:59 | [diff] [blame] | 931 | |
dpranke | a55584f1 | 2015-07-22 00:52:47 | [diff] [blame] | 932 | cmdline = [] |
Andrii Shyshkalov | c158e010 | 2018-01-10 05:52:00 | [diff] [blame] | 933 | extra_files = [ |
| 934 | '../../.vpython', |
| 935 | '../../testing/test_env.py', |
| 936 | ] |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 937 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 938 | if test_type == 'nontest': |
| 939 | self.WriteFailureAndRaise('We should not be isolating %s.' % target, |
| 940 | output_path=None) |
| 941 | |
Scott Graham | 3be4b416 | 2017-09-12 00:41:41 | [diff] [blame] | 942 | if is_android and test_type != "script": |
bpastene | e428ea9 | 2017-02-17 02:20:32 | [diff] [blame] | 943 | cmdline = [ |
John Budorick | fb97a85 | 2017-12-20 20:10:19 | [diff] [blame] | 944 | '../../testing/test_env.py', |
hzl | 9b15df5 | 2017-03-23 23:43:04 | [diff] [blame] | 945 | '../../build/android/test_wrapper/logdog_wrapper.py', |
| 946 | '--target', target, |
hzl | 9ae1445 | 2017-04-04 23:38:02 | [diff] [blame] | 947 | '--logdog-bin-cmd', '../../bin/logdog_butler', |
hzl | fc66094f | 2017-05-18 00:50:48 | [diff] [blame] | 948 | '--store-tombstones'] |
Scott Graham | 3be4b416 | 2017-09-12 00:41:41 | [diff] [blame] | 949 | elif is_fuchsia and test_type != 'script': |
John Budorick | fb97a85 | 2017-12-20 20:10:19 | [diff] [blame] | 950 | cmdline = [ |
| 951 | '../../testing/test_env.py', |
| 952 | os.path.join('bin', 'run_%s' % target), |
| 953 | ] |
kylechar | 3970568 | 2017-01-19 14:37:23 | [diff] [blame] | 954 | elif use_xvfb and test_type == 'windowed_test_launcher': |
Andrii Shyshkalov | c158e010 | 2018-01-10 05:52:00 | [diff] [blame] | 955 | extra_files.append('../../testing/xvfb.py') |
dpranke | a55584f1 | 2015-07-22 00:52:47 | [diff] [blame] | 956 | cmdline = [ |
dpranke | fe0d35e | 2016-02-05 02:43:59 | [diff] [blame] | 957 | '../../testing/xvfb.py', |
dpranke | fe0d35e | 2016-02-05 02:43:59 | [diff] [blame] | 958 | './' + str(executable) + executable_suffix, |
| 959 | '--brave-new-test-launcher', |
| 960 | '--test-launcher-bot-mode', |
| 961 | '--asan=%d' % asan, |
| 962 | '--msan=%d' % msan, |
| 963 | '--tsan=%d' % tsan, |
pcc | 46233c2 | 2017-06-20 22:11:41 | [diff] [blame] | 964 | '--cfi-diag=%d' % cfi_diag, |
dpranke | a55584f1 | 2015-07-22 00:52:47 | [diff] [blame] | 965 | ] |
| 966 | elif test_type in ('windowed_test_launcher', 'console_test_launcher'): |
dpranke | a55584f1 | 2015-07-22 00:52:47 | [diff] [blame] | 967 | cmdline = [ |
| 968 | '../../testing/test_env.py', |
dpranke | fe0d35e | 2016-02-05 02:43:59 | [diff] [blame] | 969 | './' + str(executable) + executable_suffix, |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 970 | '--brave-new-test-launcher', |
| 971 | '--test-launcher-bot-mode', |
| 972 | '--asan=%d' % asan, |
| 973 | '--msan=%d' % msan, |
| 974 | '--tsan=%d' % tsan, |
pcc | 46233c2 | 2017-06-20 22:11:41 | [diff] [blame] | 975 | '--cfi-diag=%d' % cfi_diag, |
dpranke | a55584f1 | 2015-07-22 00:52:47 | [diff] [blame] | 976 | ] |
dpranke | 6abd865 | 2015-08-28 03:21:11 | [diff] [blame] | 977 | elif test_type == 'script': |
dpranke | 6abd865 | 2015-08-28 03:21:11 | [diff] [blame] | 978 | cmdline = [ |
| 979 | '../../testing/test_env.py', |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 980 | '../../' + self.ToSrcRelPath(isolate_map[target]['script']) |
dpranke | fe0d35e | 2016-02-05 02:43:59 | [diff] [blame] | 981 | ] |
dpranke | a55584f1 | 2015-07-22 00:52:47 | [diff] [blame] | 982 | elif test_type in ('raw'): |
dpranke | a55584f1 | 2015-07-22 00:52:47 | [diff] [blame] | 983 | cmdline = [ |
| 984 | './' + str(target) + executable_suffix, |
dpranke | fe0d35e | 2016-02-05 02:43:59 | [diff] [blame] | 985 | ] |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 986 | |
dpranke | a55584f1 | 2015-07-22 00:52:47 | [diff] [blame] | 987 | else: |
| 988 | self.WriteFailureAndRaise('No command line for %s found (test type %s).' |
| 989 | % (target, test_type), output_path=None) |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 990 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 991 | cmdline += isolate_map[target].get('args', []) |
dpranke | fe0d35e | 2016-02-05 02:43:59 | [diff] [blame] | 992 | |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 993 | return cmdline, extra_files |
| 994 | |
dpranke | 74559b5 | 2015-06-10 21:20:39 | [diff] [blame] | 995 | def ToAbsPath(self, build_path, *comps): |
dpranke | 8c2cfd3 | 2015-09-17 20:12:33 | [diff] [blame] | 996 | return self.PathJoin(self.chromium_src_dir, |
| 997 | self.ToSrcRelPath(build_path), |
| 998 | *comps) |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 999 | |
dpranke | ee5b51f6 | 2015-04-09 00:03:22 | [diff] [blame] | 1000 | def ToSrcRelPath(self, path): |
| 1001 | """Returns a relative path from the top of the repo.""" |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 1002 | if path.startswith('//'): |
| 1003 | return path[2:].replace('/', self.sep) |
| 1004 | return self.RelPath(path, self.chromium_src_dir) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1005 | |
Dirk Pranke | 0fd41bcd | 2015-06-19 00:05:50 | [diff] [blame] | 1006 | def RunGNAnalyze(self, vals): |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1007 | # Analyze runs before 'gn gen' now, so we need to run gn gen |
Dirk Pranke | 0fd41bcd | 2015-06-19 00:05:50 | [diff] [blame] | 1008 | # in order to ensure that we have a build directory. |
Dirk Pranke | a3727f9 | 2017-07-17 17:30:33 | [diff] [blame] | 1009 | ret = self.RunGNGen(vals, compute_grit_inputs_for_analyze=True) |
Dirk Pranke | 0fd41bcd | 2015-06-19 00:05:50 | [diff] [blame] | 1010 | if ret: |
| 1011 | return ret |
| 1012 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1013 | build_path = self.args.path[0] |
| 1014 | input_path = self.args.input_path[0] |
| 1015 | gn_input_path = input_path + '.gn' |
| 1016 | output_path = self.args.output_path[0] |
| 1017 | gn_output_path = output_path + '.gn' |
| 1018 | |
dpranke | 7837fc36 | 2015-11-19 03:54:16 | [diff] [blame] | 1019 | inp = self.ReadInputJSON(['files', 'test_targets', |
| 1020 | 'additional_compile_targets']) |
dpranke | cda0033 | 2015-04-11 04:18:32 | [diff] [blame] | 1021 | if self.args.verbose: |
| 1022 | self.Print() |
| 1023 | self.Print('analyze input:') |
| 1024 | self.PrintJSON(inp) |
| 1025 | self.Print() |
| 1026 | |
dpranke | 7673466 | 2015-04-16 02:17:50 | [diff] [blame] | 1027 | |
dpranke | 7c5f614d | 2015-07-22 23:43:39 | [diff] [blame] | 1028 | # This shouldn't normally happen, but could due to unusual race conditions, |
| 1029 | # like a try job that gets scheduled before a patch lands but runs after |
| 1030 | # the patch has landed. |
| 1031 | if not inp['files']: |
| 1032 | self.Print('Warning: No files modified in patch, bailing out early.') |
dpranke | 7837fc36 | 2015-11-19 03:54:16 | [diff] [blame] | 1033 | self.WriteJSON({ |
| 1034 | 'status': 'No dependency', |
| 1035 | 'compile_targets': [], |
| 1036 | 'test_targets': [], |
| 1037 | }, output_path) |
dpranke | 7c5f614d | 2015-07-22 23:43:39 | [diff] [blame] | 1038 | return 0 |
| 1039 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1040 | gn_inp = {} |
dpranke | b7b183f | 2017-04-24 23:50:16 | [diff] [blame] | 1041 | gn_inp['files'] = ['//' + f for f in inp['files'] if not f.startswith('//')] |
dpranke | f61de2f | 2015-05-14 04:09:56 | [diff] [blame] | 1042 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1043 | isolate_map = self.ReadIsolateMap() |
| 1044 | err, gn_inp['additional_compile_targets'] = self.MapTargetsToLabels( |
| 1045 | isolate_map, inp['additional_compile_targets']) |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1046 | if err: |
| 1047 | raise MBErr(err) |
| 1048 | |
| 1049 | err, gn_inp['test_targets'] = self.MapTargetsToLabels( |
| 1050 | isolate_map, inp['test_targets']) |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1051 | if err: |
| 1052 | raise MBErr(err) |
| 1053 | labels_to_targets = {} |
| 1054 | for i, label in enumerate(gn_inp['test_targets']): |
| 1055 | labels_to_targets[label] = inp['test_targets'][i] |
| 1056 | |
dpranke | f61de2f | 2015-05-14 04:09:56 | [diff] [blame] | 1057 | try: |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1058 | self.WriteJSON(gn_inp, gn_input_path) |
| 1059 | cmd = self.GNCmd('analyze', build_path, gn_input_path, gn_output_path) |
| 1060 | ret, _, _ = self.Run(cmd, force_verbose=True) |
| 1061 | if ret: |
| 1062 | return ret |
dpranke | 067d014 | 2015-05-14 22:52:45 | [diff] [blame] | 1063 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1064 | gn_outp_str = self.ReadFile(gn_output_path) |
| 1065 | try: |
| 1066 | gn_outp = json.loads(gn_outp_str) |
| 1067 | except Exception as e: |
| 1068 | self.Print("Failed to parse the JSON string GN returned: %s\n%s" |
| 1069 | % (repr(gn_outp_str), str(e))) |
| 1070 | raise |
| 1071 | |
| 1072 | outp = {} |
| 1073 | if 'status' in gn_outp: |
| 1074 | outp['status'] = gn_outp['status'] |
| 1075 | if 'error' in gn_outp: |
| 1076 | outp['error'] = gn_outp['error'] |
| 1077 | if 'invalid_targets' in gn_outp: |
| 1078 | outp['invalid_targets'] = gn_outp['invalid_targets'] |
| 1079 | if 'compile_targets' in gn_outp: |
Dirk Pranke | 4516507 | 2017-11-08 04:57:49 | [diff] [blame] | 1080 | all_input_compile_targets = sorted( |
| 1081 | set(inp['test_targets'] + inp['additional_compile_targets'])) |
| 1082 | |
| 1083 | # If we're building 'all', we can throw away the rest of the targets |
| 1084 | # since they're redundant. |
dpranke | 385a310 | 2016-09-20 22:04:08 | [diff] [blame] | 1085 | if 'all' in gn_outp['compile_targets']: |
| 1086 | outp['compile_targets'] = ['all'] |
| 1087 | else: |
Dirk Pranke | 4516507 | 2017-11-08 04:57:49 | [diff] [blame] | 1088 | outp['compile_targets'] = gn_outp['compile_targets'] |
| 1089 | |
| 1090 | # crbug.com/736215: When GN returns targets back, for targets in |
| 1091 | # the default toolchain, GN will have generated a phony ninja |
| 1092 | # target matching the label, and so we can safely (and easily) |
| 1093 | # transform any GN label into the matching ninja target. For |
| 1094 | # targets in other toolchains, though, GN doesn't generate the |
| 1095 | # phony targets, and we don't know how to turn the labels into |
| 1096 | # compile targets. In this case, we also conservatively give up |
| 1097 | # and build everything. Probably the right thing to do here is |
| 1098 | # to have GN return the compile targets directly. |
| 1099 | if any("(" in target for target in outp['compile_targets']): |
| 1100 | self.Print('WARNING: targets with non-default toolchains were ' |
| 1101 | 'found, building everything instead.') |
| 1102 | outp['compile_targets'] = all_input_compile_targets |
| 1103 | else: |
dpranke | 385a310 | 2016-09-20 22:04:08 | [diff] [blame] | 1104 | outp['compile_targets'] = [ |
Dirk Pranke | 4516507 | 2017-11-08 04:57:49 | [diff] [blame] | 1105 | label.replace('//', '') for label in outp['compile_targets']] |
| 1106 | |
| 1107 | # Windows has a maximum command line length of 8k; even Linux |
| 1108 | # maxes out at 128k; if analyze returns a *really long* list of |
| 1109 | # targets, we just give up and conservatively build everything instead. |
| 1110 | # Probably the right thing here is for ninja to support response |
| 1111 | # files as input on the command line |
| 1112 | # (see https://github.com/ninja-build/ninja/issues/1355). |
| 1113 | if len(' '.join(outp['compile_targets'])) > 7*1024: |
| 1114 | self.Print('WARNING: Too many compile targets were affected.') |
| 1115 | self.Print('WARNING: Building everything instead to avoid ' |
| 1116 | 'command-line length issues.') |
| 1117 | outp['compile_targets'] = all_input_compile_targets |
| 1118 | |
| 1119 | |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1120 | if 'test_targets' in gn_outp: |
| 1121 | outp['test_targets'] = [ |
| 1122 | labels_to_targets[label] for label in gn_outp['test_targets']] |
| 1123 | |
| 1124 | if self.args.verbose: |
| 1125 | self.Print() |
| 1126 | self.Print('analyze output:') |
| 1127 | self.PrintJSON(outp) |
| 1128 | self.Print() |
| 1129 | |
| 1130 | self.WriteJSON(outp, output_path) |
| 1131 | |
dpranke | f61de2f | 2015-05-14 04:09:56 | [diff] [blame] | 1132 | finally: |
dpranke | cb4a2e24 | 2016-09-19 01:13:14 | [diff] [blame] | 1133 | if self.Exists(gn_input_path): |
| 1134 | self.RemoveFile(gn_input_path) |
| 1135 | if self.Exists(gn_output_path): |
| 1136 | self.RemoveFile(gn_output_path) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1137 | |
| 1138 | return 0 |
| 1139 | |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 1140 | def ReadInputJSON(self, required_keys): |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1141 | path = self.args.input_path[0] |
dpranke | cda0033 | 2015-04-11 04:18:32 | [diff] [blame] | 1142 | output_path = self.args.output_path[0] |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1143 | if not self.Exists(path): |
dpranke | cda0033 | 2015-04-11 04:18:32 | [diff] [blame] | 1144 | self.WriteFailureAndRaise('"%s" does not exist' % path, output_path) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1145 | |
| 1146 | try: |
| 1147 | inp = json.loads(self.ReadFile(path)) |
| 1148 | except Exception as e: |
| 1149 | self.WriteFailureAndRaise('Failed to read JSON input from "%s": %s' % |
dpranke | cda0033 | 2015-04-11 04:18:32 | [diff] [blame] | 1150 | (path, e), output_path) |
dpranke | d811358 | 2015-06-05 20:08:25 | [diff] [blame] | 1151 | |
| 1152 | for k in required_keys: |
| 1153 | if not k in inp: |
| 1154 | self.WriteFailureAndRaise('input file is missing a "%s" key' % k, |
| 1155 | output_path) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1156 | |
| 1157 | return inp |
| 1158 | |
dpranke | d5b2b943 | 2015-06-23 16:55:30 | [diff] [blame] | 1159 | def WriteFailureAndRaise(self, msg, output_path): |
| 1160 | if output_path: |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 1161 | self.WriteJSON({'error': msg}, output_path, force_verbose=True) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1162 | raise MBErr(msg) |
| 1163 | |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 1164 | def WriteJSON(self, obj, path, force_verbose=False): |
dpranke | cda0033 | 2015-04-11 04:18:32 | [diff] [blame] | 1165 | try: |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 1166 | self.WriteFile(path, json.dumps(obj, indent=2, sort_keys=True) + '\n', |
| 1167 | force_verbose=force_verbose) |
dpranke | cda0033 | 2015-04-11 04:18:32 | [diff] [blame] | 1168 | except Exception as e: |
| 1169 | raise MBErr('Error %s writing to the output path "%s"' % |
| 1170 | (e, path)) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1171 | |
aneeshm | de50f47 | 2016-04-01 01:13:10 | [diff] [blame] | 1172 | def CheckCompile(self, master, builder): |
| 1173 | url_template = self.args.url_template + '/{builder}/builds/_all?as_text=1' |
| 1174 | url = urllib2.quote(url_template.format(master=master, builder=builder), |
| 1175 | safe=':/()?=') |
| 1176 | try: |
| 1177 | builds = json.loads(self.Fetch(url)) |
| 1178 | except Exception as e: |
| 1179 | return str(e) |
| 1180 | successes = sorted( |
| 1181 | [int(x) for x in builds.keys() if "text" in builds[x] and |
| 1182 | cmp(builds[x]["text"][:2], ["build", "successful"]) == 0], |
| 1183 | reverse=True) |
| 1184 | if not successes: |
| 1185 | return "no successful builds" |
| 1186 | build = builds[str(successes[0])] |
| 1187 | step_names = set([step["name"] for step in build["steps"]]) |
| 1188 | compile_indicators = set(["compile", "compile (with patch)", "analyze"]) |
| 1189 | if compile_indicators & step_names: |
| 1190 | return "compiles" |
| 1191 | return "does not compile" |
| 1192 | |
dpranke | 3cec199c | 2015-09-22 23:29:02 | [diff] [blame] | 1193 | def PrintCmd(self, cmd, env): |
| 1194 | if self.platform == 'win32': |
| 1195 | env_prefix = 'set ' |
| 1196 | env_quoter = QuoteForSet |
| 1197 | shell_quoter = QuoteForCmd |
| 1198 | else: |
| 1199 | env_prefix = '' |
| 1200 | env_quoter = pipes.quote |
| 1201 | shell_quoter = pipes.quote |
| 1202 | |
| 1203 | def print_env(var): |
| 1204 | if env and var in env: |
| 1205 | self.Print('%s%s=%s' % (env_prefix, var, env_quoter(env[var]))) |
| 1206 | |
dpranke | ec07926 | 2016-06-07 02:21:20 | [diff] [blame] | 1207 | print_env('LLVM_FORCE_HEAD_REVISION') |
dpranke | 3cec199c | 2015-09-22 23:29:02 | [diff] [blame] | 1208 | |
dpranke | 8c2cfd3 | 2015-09-17 20:12:33 | [diff] [blame] | 1209 | if cmd[0] == self.executable: |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1210 | cmd = ['python'] + cmd[1:] |
dpranke | 3cec199c | 2015-09-22 23:29:02 | [diff] [blame] | 1211 | self.Print(*[shell_quoter(arg) for arg in cmd]) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1212 | |
dpranke | cda0033 | 2015-04-11 04:18:32 | [diff] [blame] | 1213 | def PrintJSON(self, obj): |
| 1214 | self.Print(json.dumps(obj, indent=2, sort_keys=True)) |
| 1215 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 1216 | def Build(self, target): |
| 1217 | build_dir = self.ToSrcRelPath(self.args.path[0]) |
| 1218 | ninja_cmd = ['ninja', '-C', build_dir] |
| 1219 | if self.args.jobs: |
| 1220 | ninja_cmd.extend(['-j', '%d' % self.args.jobs]) |
| 1221 | ninja_cmd.append(target) |
| 1222 | ret, _, _ = self.Run(ninja_cmd, force_verbose=False, buffer_output=False) |
| 1223 | return ret |
| 1224 | |
| 1225 | def Run(self, cmd, env=None, force_verbose=True, buffer_output=True): |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1226 | # This function largely exists so it can be overridden for testing. |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 1227 | if self.args.dryrun or self.args.verbose or force_verbose: |
dpranke | 3cec199c | 2015-09-22 23:29:02 | [diff] [blame] | 1228 | self.PrintCmd(cmd, env) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1229 | if self.args.dryrun: |
| 1230 | return 0, '', '' |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 1231 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 1232 | ret, out, err = self.Call(cmd, env=env, buffer_output=buffer_output) |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 1233 | if self.args.verbose or force_verbose: |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 1234 | if ret: |
| 1235 | self.Print(' -> returned %d' % ret) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1236 | if out: |
dpranke | ee5b51f6 | 2015-04-09 00:03:22 | [diff] [blame] | 1237 | self.Print(out, end='') |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1238 | if err: |
dpranke | ee5b51f6 | 2015-04-09 00:03:22 | [diff] [blame] | 1239 | self.Print(err, end='', file=sys.stderr) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1240 | return ret, out, err |
| 1241 | |
dpranke | 751516a | 2015-10-03 01:11:34 | [diff] [blame] | 1242 | def Call(self, cmd, env=None, buffer_output=True): |
| 1243 | if buffer_output: |
| 1244 | p = subprocess.Popen(cmd, shell=False, cwd=self.chromium_src_dir, |
| 1245 | stdout=subprocess.PIPE, stderr=subprocess.PIPE, |
| 1246 | env=env) |
| 1247 | out, err = p.communicate() |
| 1248 | else: |
| 1249 | p = subprocess.Popen(cmd, shell=False, cwd=self.chromium_src_dir, |
| 1250 | env=env) |
| 1251 | p.wait() |
| 1252 | out = err = '' |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1253 | return p.returncode, out, err |
| 1254 | |
| 1255 | def ExpandUser(self, path): |
| 1256 | # This function largely exists so it can be overridden for testing. |
| 1257 | return os.path.expanduser(path) |
| 1258 | |
| 1259 | def Exists(self, path): |
| 1260 | # This function largely exists so it can be overridden for testing. |
| 1261 | return os.path.exists(path) |
| 1262 | |
dpranke | 867bcf4a | 2016-03-14 22:28:32 | [diff] [blame] | 1263 | def Fetch(self, url): |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 1264 | # This function largely exists so it can be overridden for testing. |
dpranke | 867bcf4a | 2016-03-14 22:28:32 | [diff] [blame] | 1265 | f = urllib2.urlopen(url) |
| 1266 | contents = f.read() |
| 1267 | f.close() |
| 1268 | return contents |
| 1269 | |
dpranke | c3441d1 | 2015-06-23 23:01:35 | [diff] [blame] | 1270 | def MaybeMakeDirectory(self, path): |
| 1271 | try: |
| 1272 | os.makedirs(path) |
| 1273 | except OSError, e: |
| 1274 | if e.errno != errno.EEXIST: |
| 1275 | raise |
| 1276 | |
dpranke | 8c2cfd3 | 2015-09-17 20:12:33 | [diff] [blame] | 1277 | def PathJoin(self, *comps): |
| 1278 | # This function largely exists so it can be overriden for testing. |
| 1279 | return os.path.join(*comps) |
| 1280 | |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 1281 | def Print(self, *args, **kwargs): |
| 1282 | # This function largely exists so it can be overridden for testing. |
| 1283 | print(*args, **kwargs) |
aneeshm | de50f47 | 2016-04-01 01:13:10 | [diff] [blame] | 1284 | if kwargs.get('stream', sys.stdout) == sys.stdout: |
| 1285 | sys.stdout.flush() |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 1286 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1287 | def ReadFile(self, path): |
| 1288 | # This function largely exists so it can be overriden for testing. |
| 1289 | with open(path) as fp: |
| 1290 | return fp.read() |
| 1291 | |
dpranke | 030d7a6d | 2016-03-26 17:23:50 | [diff] [blame] | 1292 | def RelPath(self, path, start='.'): |
| 1293 | # This function largely exists so it can be overriden for testing. |
| 1294 | return os.path.relpath(path, start) |
| 1295 | |
dpranke | f61de2f | 2015-05-14 04:09:56 | [diff] [blame] | 1296 | def RemoveFile(self, path): |
| 1297 | # This function largely exists so it can be overriden for testing. |
| 1298 | os.remove(path) |
| 1299 | |
dpranke | c161aa9 | 2015-09-14 20:21:13 | [diff] [blame] | 1300 | def RemoveDirectory(self, abs_path): |
dpranke | 8c2cfd3 | 2015-09-17 20:12:33 | [diff] [blame] | 1301 | if self.platform == 'win32': |
dpranke | c161aa9 | 2015-09-14 20:21:13 | [diff] [blame] | 1302 | # In other places in chromium, we often have to retry this command |
| 1303 | # because we're worried about other processes still holding on to |
| 1304 | # file handles, but when MB is invoked, it will be early enough in the |
| 1305 | # build that their should be no other processes to interfere. We |
| 1306 | # can change this if need be. |
| 1307 | self.Run(['cmd.exe', '/c', 'rmdir', '/q', '/s', abs_path]) |
| 1308 | else: |
| 1309 | shutil.rmtree(abs_path, ignore_errors=True) |
| 1310 | |
dpranke | f61de2f | 2015-05-14 04:09:56 | [diff] [blame] | 1311 | def TempFile(self, mode='w'): |
| 1312 | # This function largely exists so it can be overriden for testing. |
| 1313 | return tempfile.NamedTemporaryFile(mode=mode, delete=False) |
| 1314 | |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 1315 | def WriteFile(self, path, contents, force_verbose=False): |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1316 | # This function largely exists so it can be overriden for testing. |
dpranke | e0547cd | 2015-09-15 01:27:40 | [diff] [blame] | 1317 | if self.args.dryrun or self.args.verbose or force_verbose: |
dpranke | d5b2b943 | 2015-06-23 16:55:30 | [diff] [blame] | 1318 | self.Print('\nWriting """\\\n%s""" to %s.\n' % (contents, path)) |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1319 | with open(path, 'w') as fp: |
| 1320 | return fp.write(contents) |
| 1321 | |
dpranke | f61de2f | 2015-05-14 04:09:56 | [diff] [blame] | 1322 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1323 | class MBErr(Exception): |
| 1324 | pass |
| 1325 | |
| 1326 | |
dpranke | 3cec199c | 2015-09-22 23:29:02 | [diff] [blame] | 1327 | # See http://goo.gl/l5NPDW and http://goo.gl/4Diozm for the painful |
| 1328 | # details of this next section, which handles escaping command lines |
| 1329 | # so that they can be copied and pasted into a cmd window. |
| 1330 | UNSAFE_FOR_SET = set('^<>&|') |
| 1331 | UNSAFE_FOR_CMD = UNSAFE_FOR_SET.union(set('()%')) |
| 1332 | ALL_META_CHARS = UNSAFE_FOR_CMD.union(set('"')) |
| 1333 | |
| 1334 | |
| 1335 | def QuoteForSet(arg): |
| 1336 | if any(a in UNSAFE_FOR_SET for a in arg): |
| 1337 | arg = ''.join('^' + a if a in UNSAFE_FOR_SET else a for a in arg) |
| 1338 | return arg |
| 1339 | |
| 1340 | |
| 1341 | def QuoteForCmd(arg): |
| 1342 | # First, escape the arg so that CommandLineToArgvW will parse it properly. |
dpranke | 3cec199c | 2015-09-22 23:29:02 | [diff] [blame] | 1343 | if arg == '' or ' ' in arg or '"' in arg: |
| 1344 | quote_re = re.compile(r'(\\*)"') |
| 1345 | arg = '"%s"' % (quote_re.sub(lambda mo: 2 * mo.group(1) + '\\"', arg)) |
| 1346 | |
| 1347 | # Then check to see if the arg contains any metacharacters other than |
| 1348 | # double quotes; if it does, quote everything (including the double |
| 1349 | # quotes) for safety. |
| 1350 | if any(a in UNSAFE_FOR_CMD for a in arg): |
| 1351 | arg = ''.join('^' + a if a in ALL_META_CHARS else a for a in arg) |
| 1352 | return arg |
| 1353 | |
| 1354 | |
dpranke | fe460231 | 2015-04-08 16:20:35 | [diff] [blame] | 1355 | if __name__ == '__main__': |
dpranke | 255085e | 2016-03-16 05:23:59 | [diff] [blame] | 1356 | sys.exit(main(sys.argv[1:])) |