Commit c6033393 authored by Takuto Ikuta's avatar Takuto Ikuta Committed by LUCI CQ

autoninja: use pure python for ninjalog_uploader_wrapper.py

I'll update to python3 separately.

Bug: b/177465438
Change-Id: Id204789f2d0854a796e5408597700d90dc8cd191
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2626863Reviewed-by: 's avatarFumitoshi Ukai <ukai@google.com>
Reviewed-by: 's avatarYoshisato Yanagisawa <yyanagisawa@google.com>
Reviewed-by: 's avatarBruce Dawson <brucedawson@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
parent 703f21e2
...@@ -24,12 +24,12 @@ if eval "$command"; then ...@@ -24,12 +24,12 @@ if eval "$command"; then
fi fi
# Collect ninjalog from googler. # Collect ninjalog from googler.
vpython "$(dirname -- "$0")/ninjalog_uploader_wrapper.py" --cmd $command python "$(dirname -- "$0")/ninjalog_uploader_wrapper.py" --cmd $command
exit exit
fi fi
# Collect ninjalog from googler. # Collect ninjalog from googler.
vpython "$(dirname -- "$0")/ninjalog_uploader_wrapper.py" --cmd $command python "$(dirname -- "$0")/ninjalog_uploader_wrapper.py" --cmd $command
# Return an error code of 1 so that if a developer types: # Return an error code of 1 so that if a developer types:
# "autoninja chrome && chrome" then chrome won't run if the build fails. # "autoninja chrome && chrome" then chrome won't run if the build fails.
......
...@@ -38,12 +38,12 @@ FOR /f "usebackq tokens=*" %%a in (`vpython %scriptdir%autoninja.py "%*"`) do ec ...@@ -38,12 +38,12 @@ FOR /f "usebackq tokens=*" %%a in (`vpython %scriptdir%autoninja.py "%*"`) do ec
REM Use call to invoke vpython script here, because we use vpython via vpython.bat. REM Use call to invoke vpython script here, because we use vpython via vpython.bat.
@if "%NINJA_SUMMARIZE_BUILD%" == "1" call vpython.bat %scriptdir%post_build_ninja_summary.py %* @if "%NINJA_SUMMARIZE_BUILD%" == "1" call vpython.bat %scriptdir%post_build_ninja_summary.py %*
@call vpython.bat %scriptdir%ninjalog_uploader_wrapper.py --cmdline %* @call python.bat %scriptdir%ninjalog_uploader_wrapper.py --cmdline %*
exit /b exit /b
:buildfailure :buildfailure
@call vpython.bat %scriptdir%ninjalog_uploader_wrapper.py --cmdline %* @call python.bat %scriptdir%ninjalog_uploader_wrapper.py --cmdline %*
REM Return an error code of 1 so that if a developer types: REM Return an error code of 1 so that if a developer types:
REM "autoninja chrome && chrome" then chrome won't run if the build fails. REM "autoninja chrome && chrome" then chrome won't run if the build fails.
......
...@@ -26,7 +26,7 @@ import subprocess ...@@ -26,7 +26,7 @@ import subprocess
import sys import sys
import time import time
import httplib2 from third_party import httplib2
# These build configs affect build performance a lot. # These build configs affect build performance a lot.
# TODO(https://crbug.com/900161): Add 'blink_symbol_level' and # TODO(https://crbug.com/900161): Add 'blink_symbol_level' and
......
#!/usr/bin/env vpython #!/usr/bin/env python
# Copyright 2018 The Chromium Authors. All rights reserved. # Copyright 2018 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be # Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file. # found in the LICENSE file.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment