build/android/pylib/buildbot_report.py - Issue 18323020: Updates the test runner script exit codes

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: build/android/pylib/buildbot_report.py

Issue 18323020: Updates the test runner script exit codes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Makes several minor fixes Created 7 years, 5 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

OLD	NEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.	1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.

2 # Use of this source code is governed by a BSD-style license that can be	2 # Use of this source code is governed by a BSD-style license that can be

3 # found in the LICENSE file.	3 # found in the LICENSE file.

4	4

5 """Helper functions to print buildbot messages."""	5 """Helper functions to print buildbot messages."""
	frankf 2013/07/08 19:13:43 Let's move this to build/android/buildbot. This co Let's move this to build/android/buildbot. This could be a follow up CL if there are many dependencies on this. gkanwar 2013/07/08 21:17:07 I'm not sure it actually makes sense to do this. T Show quoted text On 2013/07/08 19:13:43, frankf wrote: > Let's move this to build/android/buildbot. This could be a follow up CL if there > are many dependencies on this. I'm not sure it actually makes sense to do this. These functions are relevant only to files within pylib, whereas files in the buildbot folder are generally executable scripts used by buildbot. Currently the buildbot folder isn't even a Python module for this reason (I think).
6	6

7 def PrintLink(label, url):	7 def PrintLink(label, url):

8 """Adds a link with name \|label\| linking to \|url\| to current buildbot step.	8 """Adds a link with name \|label\| linking to \|url\| to current buildbot step.

9	9

10 Args:	10 Args:

11 label: A string with the name of the label.	11 label: A string with the name of the label.

12 url: A string of the URL.	12 url: A string of the URL.

13 """	13 """

14 print '@@@STEP_LINK@%s@%s@@@' % (label, url)	14 print '@@@STEP_LINK@%s@%s@@@' % (label, url)

15	15

(...skipping 21 matching lines...) Expand all Loading...
37 print '@@@STEP_FAILURE@@@'	37 print '@@@STEP_FAILURE@@@'

38	38

39	39

40 def PrintWarning():	40 def PrintWarning():

41 """Marks the current step with a warning."""	41 """Marks the current step with a warning."""

42 print '@@@STEP_WARNINGS@@@'	42 print '@@@STEP_WARNINGS@@@'

43	43

44	44

45 def PrintNamedStep(step):	45 def PrintNamedStep(step):

46 print '@@@BUILD_STEP %s@@@' % step	46 print '@@@BUILD_STEP %s@@@' % step

47

48

49 def PrintStepResultIfNeeded(options, result):

50 if result:

51 if options.buildbot_step_failure:

52 PrintError()

53 else:

54 PrintWarning()

OLD	NEW

« no previous file with comments | « build/android/pylib/browsertests/dispatch.py ('k') | build/android/pylib/constants.py » ('j') | build/android/pylib/gtest/dispatch.py » ('J')