Re: [PATCH v1] stash show: fix breakage in 1.7.3

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

 



| Added Brandon to distribution - apologies for earlier omission.

jon.

On Sat, Sep 25, 2010 at 1:32 PM, Jon Seymour <jon.seymour@xxxxxxxxx> wrote:
> The detached-stash series regressed support for
> Â git stash show stash@{0}
>
> due to a faulty assumption that:
> Â git rev-parse --no-revs -- stash@{0}
>
> would treat stash@{0} as a revision reference and
> thus not output it.
>
> This patch restores the behaviour of git stash show
> so that git rev-parse is not used for parsing flags
> and only flag like options are assigned to
> the FLAGS variable.
>
> It has been tested with Brandon Casey's improved t3903 tests.
>
> Signed-off-by: Jon Seymour <jon.seymour@xxxxxxxxx>
> ---
> Âgit-stash.sh | Â Â5 +----
> Â1 files changed, 1 insertions(+), 4 deletions(-)
>
> This revision further simplifies the parsing code
> by removing use of git rev-parse for FLAGS parsing
> altogether.
>
> diff --git a/git-stash.sh b/git-stash.sh
> index 7ce818b..8b18bb5 100755
> --- a/git-stash.sh
> +++ b/git-stash.sh
> @@ -265,9 +265,6 @@ parse_flags_and_rev()
> Â Â Â Âi_tree=
>
> Â Â Â ÂREV=$(git rev-parse --no-flags --symbolic "$@" 2>/dev/null)
> - Â Â Â FLAGS=$(git rev-parse --no-revs -- "$@" 2>/dev/null)
> -
> - Â Â Â set -- $FLAGS
>
> Â Â Â ÂFLAGS=
> Â Â Â Âwhile test $# -ne 0
> @@ -282,7 +279,7 @@ parse_flags_and_rev()
> Â Â Â Â Â Â Â Â Â Â Â Â--)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â:
> Â Â Â Â Â Â Â Â Â Â Â Â;;
> - Â Â Â Â Â Â Â Â Â Â Â *)
> + Â Â Â Â Â Â Â Â Â Â Â -*)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂFLAGS="${FLAGS}${FLAGS:+ }$1"
> Â Â Â Â Â Â Â Â Â Â Â Â;;
> Â Â Â Â Â Â Â Âesac
> --
> 1.7.2.14.g132f5.dirty
>
>
--
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]