Issue 3772002: Fixed incorrect option comment in run_remote_tests.

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

Issue 3772002: Fixed incorrect option comment in run_remote_tests. (Closed)

Created:
10 years, 2 months ago by DaleCurtis

Modified:
9 years, 7 months ago

Reviewers:
ericli, sosa

CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa

Base URL:
http://git.chromium.org/git/crosutils.git

Visibility:
Public.

Description

Fixed incorrect option comment in run_remote_tests. Autotest will not split comma separated arguments. The correct calling method for multiple arguments is to separate them by spaces and quote the whole thing. E.g., instead of: ./run_remote_tests --args arg1,arg2 It should be: ./run_remote_tests --args "arg1 arg2" See line 140 in src/third_party/autotest/files/server/autoserv_parser.py for confirmation. Change-Id: I88d65c0bd144942a856c2f95371d5a55b0a3a172 BUG= TEST=No code changes. Ran --help and --args to verify nothing dumb happened and the correctness of comment. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=15e4957

Patch Set 1 #

Created: 10 years, 2 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+2 lines, -1 line)			Patch
	M	run_remote_tests.sh	View		1 chunk	+2 lines, -1 line	0 comments	Download

Messages

Total messages: 2 (0 generated)

Expand Messages | Collapse Messages

LGTM

Expand Messages | Collapse Messages