Reviewers: Kevin Millikin,

Message:
PTAL. This should fix both issues you uncovered today.

Description:
push-to-trunk.sh: correctly detect existing branches, git-rm deleted files


Please review this at http://codereview.chromium.org/7886004/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M tools/push-to-trunk.sh


Index: tools/push-to-trunk.sh
diff --git a/tools/push-to-trunk.sh b/tools/push-to-trunk.sh
index 74edbc48fb4a6fc6a4078830e52b28b18493e88d..cff6bea34657e2a5f46b8bbd6934f2d6d91f0a3a 100755
--- a/tools/push-to-trunk.sh
+++ b/tools/push-to-trunk.sh
@@ -74,11 +74,11 @@ confirm() {
 }

 delete_branch() {
-  local MATCH=$(git branch | grep $1)
+  local MATCH=$(git branch | grep $1 | awk '{print $NF}' )
   if [ "$MATCH" == "$1" ] ; then
     confirm "Branch $1 exists, do you want to delete it?"
     if [ $? -eq 0 ] ; then
-      git branch -D $1
+      git branch -D $1 || die "Deleting branch '$1' failed."
       echo "Branch $1 deleted."
     else
       die "Can't continue. Please delete branch $1 and try again."
@@ -352,10 +352,16 @@ if [ $STEP -le 13 ] ; then
   echo ">>> Step 13: Apply squashed changes."
patch -p1 < "$PATCH_FILE" | tee >(awk '{print $NF}'
"$TOUCHED_FILES_FILE")
   [[ $? -eq 0 ]] || die "Applying the patch to trunk failed."
+  # Stage added and modified files.
   TOUCHED_FILES=$(cat "$TOUCHED_FILES_FILE")
   for FILE in $TOUCHED_FILES ; do
     git add "$FILE"
   done
+  # Stage deleted files.
+ DELETED_FILES=$(git status -s -uno --porcelain | grep "^ D" | awk '{print $NF}')
+  for FILE in $DELETED_FILES ; do
+    git rm "$FILE"
+  done
   rm -f "$PATCH_FILE"
   rm -f "$TOUCHED_FILES_FILE"
 fi


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to