Rob Mayoff <mayoff@xxxxxxx> writes: > diff --git a/contrib/subtree/git-subtree.sh b/contrib/subtree/git-subtree.sh > index 9f06571..b051600 100755 > --- a/contrib/subtree/git-subtree.sh > +++ b/contrib/subtree/git-subtree.sh > @@ -245,7 +245,10 @@ find_latest_squash() > case "$a" in > START) sq="$b" ;; > git-subtree-mainline:) main="$b" ;; > - git-subtree-split:) sub="$b" ;; > + git-subtree-split:) > + sub="$b" Why include the above line? > + sub="$(git rev-parse "$b^0")" || die "could not rev-parse split hash $b from commit $sq" This seems like odd quoting. Would not this do the same? sub="$(git rev-parse $b^0)" || die "could not rev-parse split hash $b from commit $sq" Perhaps I am missing something. > + ;; > END) > if [ -n "$sub" ]; then > if [ -n "$main" ]; then > @@ -278,7 +281,10 @@ find_existing_splits() > case "$a" in > START) sq="$b" ;; > git-subtree-mainline:) main="$b" ;; > - git-subtree-split:) sub="$b" ;; > + git-subtree-split:) > + sub="$b" And here too. > + sub="$(git rev-parse "$b^0")" || die "could not rev-parse split hash $b from commit $sq" Same as above. -David -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html