Rob Mayoff <[email protected]> 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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
