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

Side by Side 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/bin/bash 1 #!/bin/bash
2 2
3 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved. 3 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 # Wrapper scripts around cros_mark_as_stable that marks all packages as stable 7 # Wrapper scripts around cros_mark_as_stable that marks all packages as stable
8 # that have CROS_WORKON_COMMIT that is different than the current HEAD commit 8 # that have CROS_WORKON_COMMIT that is different than the current HEAD commit
9 # of the corresponding git repository. 9 # of the corresponding git repository.
10 10
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 [ -n "${head_commit}" ]; then 73 [ -n "${head_commit}" ]; then
74 info\ 74 info\
75 "HEAD ${head_commit} != CROS_WORKON_COMMIT ${egit_commit} for ${package}" 75 "HEAD ${head_commit} != CROS_WORKON_COMMIT ${egit_commit} for ${package}"
76 PACKAGE_LIST="${PACKAGE_LIST} ${package}" 76 PACKAGE_LIST="${PACKAGE_LIST} ${package}"
77 COMMIT_ID_LIST="${COMMIT_ID_LIST} ${head_commit}" 77 COMMIT_ID_LIST="${COMMIT_ID_LIST} ${head_commit}"
78 elif [[ ${head_commit} = ${egit_commit} ]]; then 78 elif [[ ${head_commit} = ${egit_commit} ]]; then
79 info "Commit id's match for ${package}" 79 info "Commit id's match for ${package}"
80 fi 80 fi
81 done 81 done
82 82
83 info "Candidate package list ${PACKAGE_LIST}" 83 if [ -n "${PACKAGE_LIST}" ] ; then
84 info "With commit id list ${COMMIT_ID_LIST}" 84 info "Candidate package list ${PACKAGE_LIST}"
85 info "With commit id list ${COMMIT_ID_LIST}"
85 86
86 ./cros_mark_as_stable --board ${FLAGS_board} -p "${PACKAGE_LIST}" \ 87 ./cros_mark_as_stable --board ${FLAGS_board} -p "${PACKAGE_LIST}" \
87 -i "${COMMIT_ID_LIST}" -t ${FLAGS_tracking_branch} commit || \ 88 -i "${COMMIT_ID_LIST}" -t ${FLAGS_tracking_branch} commit || \
88 die "Could not mark all packages as stable" 89 die "Could not mark all packages as stable"
90 else
91 info "No candidate packages to be marked"
92 fi
OLDNEW
« 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