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

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

Issue 2797773003: Save the logcat file in isolated directory. (Closed)
Patch Set: Created 3 years, 8 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 | « build/android/test_wrapper/logdog_wrapper.py ('k') | no next file » | 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 1073 matching lines...) Expand 10 before | Expand all | Expand 10 after
1084 1084
1085 if test_type == 'nontest': 1085 if test_type == 'nontest':
1086 self.WriteFailureAndRaise('We should not be isolating %s.' % target, 1086 self.WriteFailureAndRaise('We should not be isolating %s.' % target,
1087 output_path=None) 1087 output_path=None)
1088 1088
1089 if android and test_type != "script": 1089 if android and test_type != "script":
1090 cmdline = [ 1090 cmdline = [
1091 '../../build/android/test_wrapper/logdog_wrapper.py', 1091 '../../build/android/test_wrapper/logdog_wrapper.py',
1092 '--target', target, 1092 '--target', target,
1093 '--target-devices-file', '${SWARMING_BOT_FILE}', 1093 '--target-devices-file', '${SWARMING_BOT_FILE}',
1094 '--logdog-bin-cmd', '../../bin/logdog_butler'] 1094 '--logdog-bin-cmd', '../../bin/logdog_butler',
1095 '--logcat-output-file', '${ISOLATED_OUTDIR}/logcats']
1095 elif use_xvfb and test_type == 'windowed_test_launcher': 1096 elif use_xvfb and test_type == 'windowed_test_launcher':
1096 extra_files = [ 1097 extra_files = [
1097 '../../testing/test_env.py', 1098 '../../testing/test_env.py',
1098 '../../testing/xvfb.py', 1099 '../../testing/xvfb.py',
1099 ] 1100 ]
1100 cmdline = [ 1101 cmdline = [
1101 '../../testing/xvfb.py', 1102 '../../testing/xvfb.py',
1102 './' + str(executable) + executable_suffix, 1103 './' + str(executable) + executable_suffix,
1103 '--brave-new-test-launcher', 1104 '--brave-new-test-launcher',
1104 '--test-launcher-bot-mode', 1105 '--test-launcher-bot-mode',
(...skipping 453 matching lines...) Expand 10 before | Expand all | Expand 10 after
1558 # Then check to see if the arg contains any metacharacters other than 1559 # Then check to see if the arg contains any metacharacters other than
1559 # double quotes; if it does, quote everything (including the double 1560 # double quotes; if it does, quote everything (including the double
1560 # quotes) for safety. 1561 # quotes) for safety.
1561 if any(a in UNSAFE_FOR_CMD for a in arg): 1562 if any(a in UNSAFE_FOR_CMD for a in arg):
1562 arg = ''.join('^' + a if a in ALL_META_CHARS else a for a in arg) 1563 arg = ''.join('^' + a if a in ALL_META_CHARS else a for a in arg)
1563 return arg 1564 return arg
1564 1565
1565 1566
1566 if __name__ == '__main__': 1567 if __name__ == '__main__':
1567 sys.exit(main(sys.argv[1:])) 1568 sys.exit(main(sys.argv[1:]))
OLDNEW
« no previous file with comments | « build/android/test_wrapper/logdog_wrapper.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698