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

Unified Diff: buildbot/buildbot_pnacl_toolchain_tests.sh

Issue 1700503002: Make FYI bot test script exit with nonzero status if any tests fail. (Closed) Base URL: https://chromium.googlesource.com/native_client/src/native_client.git@master
Patch Set: Created 4 years, 10 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: buildbot/buildbot_pnacl_toolchain_tests.sh
diff --git a/buildbot/buildbot_pnacl_toolchain_tests.sh b/buildbot/buildbot_pnacl_toolchain_tests.sh
index 28f79843627cd06e9e8e77b466d7f68613504bf5..c21fc7761cd6650cb24131582fcad67c5e2c8b1f 100755
--- a/buildbot/buildbot_pnacl_toolchain_tests.sh
+++ b/buildbot/buildbot_pnacl_toolchain_tests.sh
@@ -39,6 +39,8 @@ export PNACL_BUILDBOT=true
# by long periods without console output.
export PNACL_VERBOSE=true
+EXIT_STATUS=0
+
clobber() {
echo @@@BUILD_STEP clobber@@@
rm -rf scons-out
@@ -51,6 +53,7 @@ clobber() {
handle-error() {
echo "@@@STEP_FAILURE@@@"
+ EXIT_STATUS=1
}
ignore-error() {
@@ -253,6 +256,7 @@ tc-test-bot() {
fi
done
+ exit $EXIT_STATUS
}
« 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