Re: [RFC/PATCH] Re: git submodule -b ... of current HEAD fails

On 12/02/2010 04:11 PM, Jens Lehmann wrote:

Nope, these lines date back to the time before I got involved in the
submodule business ... Seems like this "git checkout" was added in
March 2008 by Mark Levedahl (CCed), maybe he can shed some light on
that.

But to me your change looks good, so feel free to add:
Acked-by: Jens Lehmann<redacted>

quoted

diff --git a/git-submodule.sh b/git-submodule.sh
index 33bc41f..6242d7f 100755
--- a/git-submodule.sh
+++ b/git-submodule.sh

@@ -241,7 +241,7 @@ cmd_add()
  			# ash fails to wordsplit ${branch:+-b "$branch"...}
  			case "$branch" in
  			'') git checkout -f -q ;;
-			?*) git checkout -f -q -b "$branch" "origin/$branch" ;;
+			?*) git checkout -f -q -B "$branch" "origin/$branch" ;;
  			esac
  		) || die "Unable to checkout submodule '$path'"
  	fi

These lines were actually added by Ben Jackson in commit ea10b60c91 in 
2009, long after I last touched that module.

Mark