I am sorry I neglected to follow-up on this.
greened@xxxxxxxxxxxxx (David A. Greene) writes:
> 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?
My bad. Missed the diff markers. This is fine.
>> + 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.
Jeff explained it, so this is ok.
This all looks good to me. Junio, please apply.
-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