Re: [PATCH] git-rebase: Teach rebase "-" shorthand.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Brian Gesiak <modocache@xxxxxxxxx> writes:

> Teach rebase the same shorthand as checkout and merge; that is, that "-"
> means "the branch we were previously on".
>
> Reported-by: Tim Chase <git@xxxxxxxxxxxxxxxxx>
> Signed-off-by: Brian Gesiak <modocache@xxxxxxxxx>
> ---
>  git-rebase.sh     | 4 ++++
>  t/t3400-rebase.sh | 6 ++++++
>  2 files changed, 10 insertions(+)
>
> diff --git a/git-rebase.sh b/git-rebase.sh
> index 5f6732b..2c75e9f 100755
> --- a/git-rebase.sh
> +++ b/git-rebase.sh
> @@ -453,6 +453,10 @@ then
>  		test "$fork_point" = auto && fork_point=t
>  		;;
>  	*)	upstream_name="$1"
> +		if test "$upstream_name" = "-"
> +		then
> +			upstream_name="@{-1}"
> +		fi
>  		shift
>  		;;
>  	esac
> diff --git a/t/t3400-rebase.sh b/t/t3400-rebase.sh
> index 6d94b1f..00aba9f 100755
> --- a/t/t3400-rebase.sh
> +++ b/t/t3400-rebase.sh
> @@ -88,6 +88,12 @@ test_expect_success 'rebase from ambiguous branch name' '
>  	git rebase master
>  '
>  
> +test_expect_success 'rebase using shorthand' '
> +	git checkout master
> +	git checkout -b shorthand HEAD^
> +	GIT_TRACE=1 git rebase -

I'd rather not to see that TRACE there.  We would also want to make
sure the result is what we expect to see, not only the command does
not error out, no?

> +'
> +
>  test_expect_success 'rebase a single mode change' '
>  	git checkout master &&
>  	git branch -D topic &&
--
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




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]