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

Issue 10912072: [android] Remove extra adb newlines from buildbot log (Closed)

Created:
8 years, 3 months ago by Sami (do not use)
Modified:
8 years, 3 months ago
Reviewers:
bulach, Isaac (away), Sami
CC:
chromium-reviews, peter+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy+watch_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

[android] Remove extra adb newlines from buildbot log Pexpect runs adb within a pseudo-tty device (see http://www.noah.org/wiki/pexpect), so any '\n' printed by adb is written as '\r\n' to the logfile. Since adb already uses '\r\n' to terminate lines, the log ends up having '\r\r\n' at the end of each line. When this is rendered within a <pre> element in the buildbot log, an empty line is shown between each actual log line. This patch replaces the above with a single '\n' in the logfile stream, making the buildbot log easier to read. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=154760

Patch Set 1 #

Total comments: 3

Patch Set 2 : Fixed nits #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -0 lines) Patch
M build/android/pylib/android_commands.py View 1 1 chunk +22 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Sami
8 years, 3 months ago (2012-09-04 10:23:56 UTC) #1
bulach
lgtm, thanks! just nits: https://chromiumcodereview.appspot.com/10912072/diff/1/build/android/pylib/android_commands.py File build/android/pylib/android_commands.py (right): https://chromiumcodereview.appspot.com/10912072/diff/1/build/android/pylib/android_commands.py#newcode711 build/android/pylib/android_commands.py:711: class NewLineNormalizer: nit: (object) https://chromiumcodereview.appspot.com/10912072/diff/1/build/android/pylib/android_commands.py#newcode712 ...
8 years, 3 months ago (2012-09-04 11:22:56 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/skyostil@google.com/10912072/4001
8 years, 3 months ago (2012-09-04 12:34:26 UTC) #3
commit-bot: I haz the power
Try job failure for 10912072-4001 (retry) on linux_rel for step "interactive_ui_tests" (clobber build). It's a ...
8 years, 3 months ago (2012-09-04 14:16:52 UTC) #4
Isaac (away)
8 years, 3 months ago (2012-09-05 08:22:05 UTC) #5
This is awesome!  Thanks Sami!

Powered by Google App Engine
This is Rietveld 408576698