Refactor with fixes for msys-git.

git-crup

 #!/bin/bash
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # A convenience script to largely replicate the behavior of `gclient sync` in a
 # submodule-based checkout.  Fetches latest commits for top-level solutions;
 # updates submodules; and runs post-sync hooks.
 
-# Auto-update depot_tools
-if [ -z "$GIT_CRUP_REINVOKE" ]; then
-  update_depot_tools || exit 1
-  GIT_CRUP_REINVOKE=1 exec bash "$0" "$@"
-fi
-
-export GIT_MERGE_AUTOEDIT=0
-
+orig_args="$@"
 ECHO=
 pull=pull
 pull_args=
 hooks=yes
 j=10
 crup_runner="crup-runner.sh"
-runhooks="git-runhooks"
-
-kernel_name=$(uname -s)
-if [ "${kernel_name:0:5}" = "MINGW" -o "${kernel_name:0:6}" = "CYGWIN" ]; then
-  GIT_EXE=git.exe
-else
-  GIT_EXE=git
-fi
-export GIT_EXE
-
-if ( echo test | xargs --max-lines=1 true 2>/dev/null ); then
-  max_lines="--max-lines=1"
-else
-  max_lines="-L 1"
-fi
-
-if ( echo test | xargs -I bar true 2>/dev/null ); then
-  replace_arg="-I replace_arg"
-else
-  replace_arg="-ireplace_arg"
-fi
 
 usage() {
   cat <<EOF
 Usage: git-crup [-n|--dry-run] [--fetch|--sync] [-j|--jobs [jobs]]
     [--no-hooks] [<args to git-pull or git-fetch>]
+
+    -n, --dry-run        Don't do anything; just show what would have been done.
+    --fetch              Run 'git fetch' on top-level sources, but don't merge.
+    --sync               Don't do anything at all to the top-level sources.
+    -j, --jobs           Run this many jobs in parallel.
+    --no-hooks           Don't run hooks (e.g., to generate build files) after
+                         updating.
 EOF
 }
 
 serial_update() {
   ( cd "$1"
     if test -n "$toplevel_cmd"; then
-      $toplevel_cmd | sed "s/^/[$1] /g"
+      $ECHO $toplevel_cmd | sed "s/^/[$1] /g"
       if [ $? -ne 0 ]; then
         return $?
       fi
     fi
-    $GIT_EXE submodule --quiet sync
-    $GIT_EXE ls-files -s | grep ^160000 | awk '{print $4}' |
+    $ECHO $GIT_EXE submodule --quiet sync
+    $ECHO $GIT_EXE ls-files -s | grep ^160000 | awk '{print $4}' |
     while read submod; do
-      "$crup_runner" "$1/$submod"
+      $ECHO "$crup_runner" "$1/$submod"
     done
   )
 }
 
 while test $# -ne 0; do
   case "$1" in
     -j[0-9]*)
       j=$(echo "$1" | cut -c3-)
       ;;
     --jobs=[0-9]*)
@@ skipping 27 matching lines @@
       hooks=no
       ;;
     *)
       pull_args="$pull_args $1"
       break
       ;;
   esac
   shift
 done
 
+kernel_name=$(uname -s)
+if [ "${kernel_name:0:5}" = "MINGW" -o "${kernel_name:0:6}" = "CYGWIN" ]; then

[iannucci, 2013/03/19 22:11:52]

Why not do `which git` and fall back to git.exe?

[szager1, 2013/03/20 21:47:01]

On further examination, I don't think GIT_EXE is even necessary.  I was
concerned before that running 'git' on Windows would go through the whole
wrapper stack, but that doesn't appear to be the case.  So, I have just
s/GIT_EXE/git/ everywhere.

+  GIT_EXE=git.exe
+else
+  GIT_EXE=git
+fi
+
+# Auto-update depot_tools
+if [ -z "$GIT_CRUP_REINVOKE" ]; then
+  if [ "${kernel_name:0:5}" = "MINGW" ]; then
+    # Windows behaves badly if you rewrite a file while it's being executed,
+    # which might happen when update_depot_tools runs.
+    cat <<EOF > "$TEMP/git_crup_reinvoke.sh"
+#!/bin/bash
+
+cmd '/C update_depot_tools.bat'
+GIT_CRUP_REINVOKE=1 exec bash "$0" $orig_args
+EOF
+    exec bash "$TEMP/git_crup_reinvoke.sh"
+  else
+    update_depot_tools
+    GIT_CRUP_REINVOKE=1 exec bash "$0" $orig_args

[iannucci, 2013/03/19 22:11:52]

Why not just do the windows-path on all platforms? Less likely to have a
platform specific breakage and it can't hurt anything...

[szager1, 2013/03/20 21:47:01]

I separated out copying off upate_depot_tools.bat to this CL:

https://codereview.chromium.org/12755033/

This script will now run from a temporary bash script on all platforms.

+  fi
+fi
+
 while test "$PWD" != "/"; do
   if test -f "$PWD/src/.gitmodules"; then
     break
   fi
   cd ..
 done
 if test "$PWD" = "/"; then
   echo "Could not find the root of your checkout; aborting." 1>&2
   exit 1
 fi
 
+export GIT_EXE
+export GIT_MERGE_AUTOEDIT=no
+
+if ( echo test | xargs --max-lines=1 true 2>/dev/null ); then
+  max_lines="--max-lines=1"
+else
+  max_lines="-L 1"
+fi
+
+if ( echo test | xargs -I bar true 2>/dev/null ); then
+  replace_arg="-I replace_arg"
+else
+  replace_arg="-ireplace_arg"
+fi
+
 if ( echo test test | xargs -P 2 true 2>/dev/null ); then
   xargs_parallel=yes
 else
   if test "$j" != "1"; then
     echo "Warning: parallel execution is not supported on this platform." 1>&2
   fi
   xargs_parallel=no
 fi
 
 if test -n "$pull"; then
   toplevel_cmd="$GIT_EXE $pull $pull_args -q origin"
 else
   toplevel_cmd=
 fi
 
 set -o pipefail
 if test "$xargs_parallel" = "yes"; then
   ( ls -d */.git | sed 's/\/\.git$//' |
    xargs $max_lines $replace_arg -P "$j" \
-      "$crup_runner" replace_arg $toplevel_cmd |
-   xargs $max_lines -P "$j" "$crup_runner" )
+       "$crup_runner" replace_arg $ECHO $toplevel_cmd |
+   xargs $max_lines -P "$j" $ECHO "$crup_runner" )
 else 
   ls -d */.git |
   while read gitdir; do
     serial_update "${gitdir%%/.git}"
   done
 fi
 
 status=$?
 
 if [ "$status" -ne 0 ]; then
   cat 1>&2 <<EOF
 Please check the preceding terminal output for error messages.
 Run 'git submodule status' to see the current state of submodule checkouts.
 EOF
   exit $status
 fi
 
 if [ "$hooks" = "yes" ]; then
-  "$runhooks"
+  $ECHO $GIT_EXE runhooks
   status=$?
 fi
 
 echo
 exit $status