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

Unified Diff: src/scripts/build_autotest.sh

Issue 1004003: Add a blacklist to disable autotests that don't build (Closed)
Patch Set: updated wrt comments Created 10 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/scripts/build_autotest.sh
diff --git a/src/scripts/build_autotest.sh b/src/scripts/build_autotest.sh
index acb610a969642f149b8d0227d69f29f39d645c8c..44d32595e904a39b7d944cf37c57a5c7eae0eae4 100755
--- a/src/scripts/build_autotest.sh
+++ b/src/scripts/build_autotest.sh
@@ -37,14 +37,30 @@ check_board
# build default pre-compile client tests list.
ALL_TESTS="compilebench,dbench,disktest,ltp,netperf2,unixbench"
-for SITE_TEST in ../third_party/autotest/files/client/site_tests/*
+CLIENT_TEST_PATH="../third_party/autotest/files/client/site_tests"
+for SITE_TEST in ${CLIENT_TEST_PATH}/*
do
if [ -d ${SITE_TEST} ]
then
- ALL_TESTS="${ALL_TESTS},${SITE_TEST:48}"
+ ALL_TESTS="${ALL_TESTS},${SITE_TEST##${CLIENT_TEST_PATH}/}"
fi
done
+# Load the overlay specific blacklist and remove any matching tests.
+PRIMARY_BOARD_OVERLAY="${SRC_ROOT}/overlays/overlay-${FLAGS_board}"
+BLACKLIST_FILE="${PRIMARY_BOARD_OVERLAY}/autotest-blacklist"
+if [ -r "${BLACKLIST_FILE}" ]
+then
+ BLACKLISTED_TESTS=$(cat ${BLACKLIST_FILE})
+
+ for TEST in ${BLACKLISTED_TESTS}
petkov 2010/03/17 17:27:49 Another way to do this, not necessarily better or
+ do
+ ALL_TESTS=${ALL_TESTS/#${TEST},/} # match first test (test,...)
+ ALL_TESTS=${ALL_TESTS/,${TEST},/,} # match middle tests (...,test,...)
+ ALL_TESTS=${ALL_TESTS/%,${TEST}/} # match last test (...,test)
+ done
+fi
+
if [ ${FLAGS_build} == ${DEFAULT_TESTS_LIST} ]
then
if [ ${FLAGS_prompt} -eq ${FLAGS_TRUE} ]
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698