Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1017)

Side by Side Diff: tools/mb/mb.py

Issue 2187393002: Flip two Mac ASAN straggler builders to GN and fix mb audit. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | tools/mb/mb_config.pyl » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2015 The Chromium Authors. All rights reserved. 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 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """MB - the Meta-Build wrapper around GYP and GN 6 """MB - the Meta-Build wrapper around GYP and GN
7 7
8 MB is a wrapper script for GYP and GN that can be used to generate build files 8 MB is a wrapper script for GYP and GN that can be used to generate build files
9 for sets of canned configurations and analyze them. 9 for sets of canned configurations and analyze them.
10 """ 10 """
(...skipping 434 matching lines...) Expand 10 before | Expand all | Expand 10 after
445 'client.v8.fyi', 445 'client.v8.fyi',
446 'tryserver.v8', 446 'tryserver.v8',
447 ) 447 )
448 if master in MASTERS_TO_SKIP: 448 if master in MASTERS_TO_SKIP:
449 # Skip these bots because converting them is the responsibility of 449 # Skip these bots because converting them is the responsibility of
450 # those teams and out of scope for the Chromium migration to GN. 450 # those teams and out of scope for the Chromium migration to GN.
451 self.Print(' Skipped (out of scope)') 451 self.Print(' Skipped (out of scope)')
452 self.Print('') 452 self.Print('')
453 continue 453 continue
454 454
455 INTERNAL_MASTERS = ('official.desktop', 'official.desktop.continuous') 455 INTERNAL_MASTERS = ('official.desktop', 'official.desktop.continuous',
456 'internal.client.kitchensync')
456 if master in INTERNAL_MASTERS and not self.args.internal: 457 if master in INTERNAL_MASTERS and not self.args.internal:
457 # Skip these because the servers aren't accessible by default ... 458 # Skip these because the servers aren't accessible by default ...
458 self.Print(' Skipped (internal)') 459 self.Print(' Skipped (internal)')
459 self.Print('') 460 self.Print('')
460 continue 461 continue
461 462
462 try: 463 try:
463 # Fetch the /builders contents from the buildbot master. The 464 # Fetch the /builders contents from the buildbot master. The
464 # keys of the dict are the builder names themselves. 465 # keys of the dict are the builder names themselves.
465 json_contents = self.Fetch(url) 466 json_contents = self.Fetch(url)
(...skipping 1035 matching lines...) Expand 10 before | Expand all | Expand 10 after
1501 # Then check to see if the arg contains any metacharacters other than 1502 # Then check to see if the arg contains any metacharacters other than
1502 # double quotes; if it does, quote everything (including the double 1503 # double quotes; if it does, quote everything (including the double
1503 # quotes) for safety. 1504 # quotes) for safety.
1504 if any(a in UNSAFE_FOR_CMD for a in arg): 1505 if any(a in UNSAFE_FOR_CMD for a in arg):
1505 arg = ''.join('^' + a if a in ALL_META_CHARS else a for a in arg) 1506 arg = ''.join('^' + a if a in ALL_META_CHARS else a for a in arg)
1506 return arg 1507 return arg
1507 1508
1508 1509
1509 if __name__ == '__main__': 1510 if __name__ == '__main__':
1510 sys.exit(main(sys.argv[1:])) 1511 sys.exit(main(sys.argv[1:]))
OLDNEW
« no previous file with comments | « no previous file | tools/mb/mb_config.pyl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698