crosutils: Make run_remote_tests' output for ambiguous case more clear.

BUG=5817
TEST=run_remote_tests --remote=$IP control shows ^...$ on all lines.

Change-Id: Ie0c4ac16493dc97044ec0bd0fd6d6e53a69c9041

Review URL: http://codereview.chromium.org/3473015
This commit is contained in:
Ken Mixter 2010-09-23 17:12:06 -07:00
parent 853d70bae1
commit aa5504fdec

View File

@ -111,7 +111,8 @@ function main() {
FLAGS "$@" || exit 1 FLAGS "$@" || exit 1
if [[ -z "${FLAGS_ARGV}" ]]; then if [[ -z "${FLAGS_ARGV}" ]]; then
echo "Please specify tests to run. For example:" echo "Usage: $0 --remote=[hostname] [regexp...]:"
echo "Each regexp pattern must uniquely match a control file. For example:"
echo " $0 --remote=MyMachine BootPerfServer" echo " $0 --remote=MyMachine BootPerfServer"
exit 1 exit 1
fi fi
@ -185,15 +186,11 @@ function main() {
fi fi
local matches=$(echo "${finds}" | wc -l) local matches=$(echo "${finds}" | wc -l)
if [[ ${matches} -gt 1 ]]; then if [[ ${matches} -gt 1 ]]; then
echo "" echo ">>> \"${test_request}\" is an ambiguous pattern. Disambiguate by" \
echo_color "red" \ "passing one of these patterns instead:"
">>> \"${test_request}\" is ambiguous. These control file paths match:"
for FIND in ${finds}; do for FIND in ${finds}; do
echo_color "red" " * " "${FIND}" echo " ^${FIND}\$"
done done
echo ""
echo ">>> Disambiguate by copy-and-pasting the whole path above" \
"instead of passing \"${test_request}\"."
exit 1 exit 1
fi fi
for i in $(seq 1 $FLAGS_iterations); do for i in $(seq 1 $FLAGS_iterations); do