Chromium Code Reviews| Index: build/android/test_wrapper/logdog_wrapper.py |
| diff --git a/build/android/test_wrapper/logdog_wrapper.py b/build/android/test_wrapper/logdog_wrapper.py |
| index 5213cb367d38708842f8f719dc43cb09b3af13d7..f181d50e602455d0e567a3162a12feec64de8575 100755 |
| --- a/build/android/test_wrapper/logdog_wrapper.py |
| +++ b/build/android/test_wrapper/logdog_wrapper.py |
| @@ -57,9 +57,8 @@ def main(): |
| '--logcat-output-file', |
| (args.logcat_output_file if args.logcat_output_file |
| else os.path.join(logcat_output_dir, 'logcats')), |
| - '--upload-logcats-file', |
| '--target-devices-file', args.target_devices_file, |
| - '-v'] + extra_cmd_args |
| + '-v'] |
| with tempfile_ext.NamedTemporaryDirectory( |
| prefix='tmp_android_logdog_wrapper') as temp_directory: |
| @@ -67,7 +66,9 @@ def main(): |
| logging.error( |
| 'Logdog binary %s unavailable. Unable to create logdog client', |
| args.logdog_bin_cmd) |
| + test_cmd += extra_cmd_args |
|
jbudorick
2017/04/24 22:02:51
Can we move this below the if/else block and just
|
| else: |
| + test_cmd += ['--upload-logcats-file'] + extra_cmd_args |
| streamserver_uri = 'unix:%s' % os.path.join(temp_directory, |
| 'butler.sock') |
| prefix = os.path.join('android', 'swarming', 'logcats', |