Thomas Gummerer <t.gumme...@gmail.com> writes:

> On 02/17, Junio C Hamano wrote:
>> Thomas Gummerer <t.gumme...@gmail.com> writes:
>> 
>> [some people may see this message twice, as I forgot to check if the
>> copy I received had "Some A . Body" not enclosed in dq; blindly
>> doing "Reply-All" ended up listing an invalid address on my Cc: line
>> and dropped by vger. apologies]
>> 
>> > diff --git a/git-stash.sh b/git-stash.sh
>> > index a184b1e274..1446fbe2e8 100755
>> > --- a/git-stash.sh
>> > +++ b/git-stash.sh
>> > @@ -67,51 +67,20 @@ create_stash () {
>> >            case "$1" in
>> >            -m|--message)
>> >                    shift
>> > -                  test -z ${1+x} && usage
>> > -                  stash_msg="$1"
>> > -                  new_style=t
>> > +                  stash_msg=${1-"BUG: create_stash () -m requires an 
>> > argument"}
>> >                    ;;
>> 
>> Did you mean ${1?"BUG: ..."} here and also "-u" below?
>
> Yeah, shell scripts are still confusing me sometimes.  Thanks for
> catching.  Would you mind fixing this up while queuing or do you want
> me to resend?

I'll fix it up myself (please remind me if you notice that I forgot ;-).

Reply via email to