summaryrefslogtreecommitdiff
path: root/tools/warn/cpp_warn_patterns.py
diff options
context:
space:
mode:
author Tao Bao <tbao@google.com> 2019-08-02 13:24:19 -0700
committer Tao Bao <tbao@google.com> 2019-08-02 13:36:49 -0700
commit718faed3dcd7663075aee6664b1282d464efd95b (patch)
treee5310cf112095a8aa8c735e1783cb774bddef2bf /tools/warn/cpp_warn_patterns.py
parentc016c281c1f34aa2d9507cc8f2174b3ce52f4cbb (diff)
releasetools: Don't write outputs if not in verbose mode.
The change in [1] allows brillo_update_payload to write to stdout/stderr directly without buffering, to ensure dumping progress to screen continously. However, it now unconditionally writes to stdout/stderr even if caller doesn't specify `-v`. The behavior upsets PythonUnitTestResultParser in [2], which doesn't want any unexpected output from stderr for result parsing. This CL updates the logic to only give continous outputs in verbose mode. [1] commit 2f7e11ef27bc00108088eb2b0f5a5d9cc605afdd [2] https://android.googlesource.com/platform/tools/tradefederation/+/refs/heads/master/test_framework/com/android/tradefed/testtype/PythonUnitTestResultParser.java Test: Run ota_from_target_files.py with `-v`. Check there's continous output from brillo_update_payload. Test: `python -m unittest -v test_ota_from_target_files > /dev/null` doesn't show outputs from brillo_update_payload. Change-Id: I3b851203ef011a5b4d982948cf52793f6fb6ad2d
Diffstat (limited to 'tools/warn/cpp_warn_patterns.py')
0 files changed, 0 insertions, 0 deletions