Subject: Re: bug#18531: [PATCH] man: Fix an inverted test that prevented dummy-man from working

From: Andreas Schwab <[email-masked]>
Date: Mon, 22 Sep 2014 22:11:53 +0200

Pádraig Brady <[email-masked]> writes:

> On 09/22/2014 10:53 AM, Alban Bedel wrote:
>> The test for the argument count was broken, the upper limit test used
>> less-or-equal 1 instead of greater than 1.
>> ---
>>  man/dummy-man | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/man/dummy-man b/man/dummy-man
>> index bf31912..65b85d5 100755
>> --- a/man/dummy-man
>> +++ b/man/dummy-man
>> @@ -39,7 +39,7 @@ while test $# -gt 0; do
>>  done
>>  
>>  test $# -gt 0 || fatal_ "missing argument"
>> -test $# -le 1 || fatal_ "too many non-option arguments"
>> +test $# -gt 1 || fatal_ "too many non-option arguments"
>>  
>>  baseout=`basename_ "$output"`
>>  sed 's/^/WARNING: /' >&2 <<END
>
> Nice one.

Huh?

$ man/dummy-man foo
man/dummy-man: too many non-option arguments

Andreas.

-- 
Andreas Schwab, [email-masked]
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."