Thomas Gummerer <t.gummerer@xxxxxxxxx> writes:
> On 02/17, Junio C Hamano wrote:
>> Thomas Gummerer <t.gummerer@xxxxxxxxx> 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 ;-).