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

Unified Diff: cros_mark_all_as_stable

Issue 3179004: Exit cleanly if nothing to mark (Closed) Base URL: ssh://git@chromiumos-git//crosutils.git
Patch Set: Created 10 years, 4 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: cros_mark_all_as_stable
diff --git a/cros_mark_all_as_stable b/cros_mark_all_as_stable
index 1cecd4529e1f6d49c052ea9d56ebc10011396d0f..fab9c8e19bae14a8883b0658e1fb2162e417d282 100755
--- a/cros_mark_all_as_stable
+++ b/cros_mark_all_as_stable
@@ -80,9 +80,13 @@ for package in ${PACKAGES}; do
fi
done
-info "Candidate package list ${PACKAGE_LIST}"
-info "With commit id list ${COMMIT_ID_LIST}"
+if [ -n "${PACKAGE_LIST}" ] ; then
+ info "Candidate package list ${PACKAGE_LIST}"
+ info "With commit id list ${COMMIT_ID_LIST}"
-./cros_mark_as_stable --board ${FLAGS_board} -p "${PACKAGE_LIST}" \
- -i "${COMMIT_ID_LIST}" -t ${FLAGS_tracking_branch} commit || \
+ ./cros_mark_as_stable --board ${FLAGS_board} -p "${PACKAGE_LIST}" \
+ -i "${COMMIT_ID_LIST}" -t ${FLAGS_tracking_branch} commit || \
die "Could not mark all packages as stable"
+else
+ info "No candidate packages to be marked"
+fi
« 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