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

Issue 6346005: Modified the README.txt to include information about call_autoserv.py script.

Created:
9 years, 11 months ago by ruchic
Modified:
9 years, 11 months ago
Reviewers:
kmixter1, sosa
CC:
krisr1, rchenna
Visibility:
Public.

Description

Modified the README.txt to include information about call_autoserv.py script. Change-Id: I9c61549fa332123b6cef4afc76ae14c212a37b42 BUG=chromium-os:10197 TEST=Go through the README.txt to make sure that information is sufficient.

Patch Set 1 #

Patch Set 2 : Correction to usage info for call_autoserv.py parameters. #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+98 lines, -10 lines) Patch
M client/site_tests/suite_HWQual/README.txt View 1 9 chunks +98 lines, -10 lines 2 comments Download

Messages

Total messages: 2 (0 generated)
ruchic
Please review this CL with changes made to suite_HWQual/README.txt file to provide usage information for ...
9 years, 11 months ago (2011-01-18 23:51:41 UTC) #1
kmixter1
9 years, 11 months ago (2011-01-21 20:12:11 UTC) #2
http://codereview.chromium.org/6346005/diff/2001/client/site_tests/suite_HWQu...
File client/site_tests/suite_HWQual/README.txt (right):

http://codereview.chromium.org/6346005/diff/2001/client/site_tests/suite_HWQu...
client/site_tests/suite_HWQual/README.txt:108: $ python call_autoserv.py
--ip=<DUT_IP> --test=auto
Why explicitly call python?

http://codereview.chromium.org/6346005/diff/2001/client/site_tests/suite_HWQu...
client/site_tests/suite_HWQual/README.txt:122: Alternate way to run the HWQual
tests
I suggest removing this and all the autoserv direct calls below.  Let's just
support this new, better way - but you'll need to be support it. :)

Powered by Google App Engine
This is Rietveld 408576698