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

Unified Diff: cros_workon

Issue 3335004: Whitespace/tab cleanup for cros_workon (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/crosutils.git
Patch Set: Created 10 years, 3 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_workon
diff --git a/cros_workon b/cros_workon
index e9400da47dc31b26289c40f44d3a58100aaeada3..d23756786d89a8be46bce970b4efa4dfd3fdd8a9 100755
--- a/cros_workon
+++ b/cros_workon
@@ -150,22 +150,22 @@ regen_manifest_and_sync() {
# ensure we don't clobber existing manifest entries
if [ -f "${local_manifest}" ]; then
- grep -q -i "<remote" "${local_manifest}"
- if [ $? -ne 1 ]; then
- die "Your local manifest will be clobbered running cros_workon. You can not have any <remote> tags at all."
- fi
- sed -ne '/^[[:space:]]\+[^[:space:]]/q1' "${local_manifest}"
- if [ $? -ne 0 ]; then
- die "Your local manifest will be clobbered running cros_workon. You can not have any tags that span multiple lines or are indented."
- fi
- egrep -q "^[^<]" "${local_manifest}"
- if [ $? -ne 1 ]; then
- die "Your local manifest will be clobbered running cros_workon. You can not have any tags that span multiple lines"
- fi
- grep -v "<project" "${local_manifest}" | grep -q -i "<project"
- if [ $? -eq 0 ]; then
- die "Your local manifest has mixed case tags for project. You can not have mixed case tags"
- fi
+ grep -q -i "<remote" "${local_manifest}"
+ if [ $? -ne 1 ]; then
+ die "Your local manifest will be clobbered running cros_workon. You can not have any <remote> tags at all."
+ fi
+ sed -ne '/^[[:space:]]\+[^[:space:]]/q1' "${local_manifest}"
+ if [ $? -ne 0 ]; then
+ die "Your local manifest will be clobbered running cros_workon. You can not have any tags that span multiple lines or are indented."
+ fi
+ egrep -q "^[^<]" "${local_manifest}"
+ if [ $? -ne 1 ]; then
+ die "Your local manifest will be clobbered running cros_workon. You can not have any tags that span multiple lines"
+ fi
+ grep -v "<project" "${local_manifest}" | grep -q -i "<project"
+ if [ $? -eq 0 ]; then
+ die "Your local manifest has mixed case tags for project. You can not have mixed case tags"
+ fi
fi
# preserve old manifest entries
[ -f "${local_manifest}" ] && \
« 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