For example: https://logs.chromium.org/v/?s=chromium%2Fbb%2Fchromium.clang%2FToTLinuxCoverage%2F14%2F%2B%2Frecipes%2Fsteps%2Fread_log_output_of_url_unittests%2F0%2Flogs%2Furl_unittests_output.log%2F0 This needs to be fixed.
Yes, definitely :(
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build/+/b35c66bfc5e92c8d6c212500c1b60bd314da418b commit b35c66bfc5e92c8d6c212500c1b60bd314da418b Author: Yuke Liao <liaoyuke@chromium.org> Date: Wed May 23 18:50:39 2018 Coverage: Fix a bug on Clang ToT coverage builders. The read log output of test targets steps are printing one character per line, and this CL fixes it to print the output by lines. Bug: 845552 Change-Id: I14f8a9a0f6af352f7530c7a2044aae3c01f8dd2c Reviewed-on: https://chromium-review.googlesource.com/1069981 Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> [modify] https://crrev.com/b35c66bfc5e92c8d6c212500c1b60bd314da418b/scripts/slave/recipes/chromium_clang_coverage_tot.expected/full_chromium_clang_ToTLinuxCoverage.json [modify] https://crrev.com/b35c66bfc5e92c8d6c212500c1b60bd314da418b/scripts/slave/recipes/chromium_clang_coverage_tot.expected/full_chromium_clang_ToTMacCoverage.json [modify] https://crrev.com/b35c66bfc5e92c8d6c212500c1b60bd314da418b/scripts/slave/recipes/chromium_clang_coverage_tot.py
Comment 1 by infe...@chromium.org
, May 22 2018Status: Assigned (was: Untriaged)