From cf7e8ff5b7841ab950c2cbfab15ac736f2f8edab Mon Sep 17 00:00:00 2001 From: Chris Sosa Date: Wed, 11 Aug 2010 16:23:33 -0700 Subject: [PATCH] Exit cleanly if nothing to mark TEST=Ran cros_mark_all_as_stable with packages and without packages to mark and had them both succeed with exit code 0. BUG=5486 Review URL: http://codereview.chromium.org/3179004 --- cros_mark_all_as_stable | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/cros_mark_all_as_stable b/cros_mark_all_as_stable index 1cecd4529e..fab9c8e19b 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