This patch has been superceded by:
[PATCH] stash: simplify parsing fixes
which applies on the Brian's fix that has been applied to maint.
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