Re: banner(1): fix SYNOPSIS

2015-09-26 Thread Benny Lofgren
On 2015-09-24 08:11, Jason McIntyre wrote:
> On Wed, Sep 23, 2015 at 07:42:54PM -0400, Michael Reed wrote:
>> banner(1) without arguments works fine, so denote it as such
>> in the manual.
>>
> 
> in the context of banner(1), define "works fine". i mean, technically
> maybe it does but, practically, why try to show this to the reader? is
> there a use for banner without arguments?

For one thing, it is thereby "script-safe". In other words, you can do
things like banner $mybanner and safely rely on it silently working
without emitting error messages even if $mybanner is empty.


Regards,
/Benny


>>
>>
>> Index: src/usr.bin/banner/banner.1
>> ===
>> RCS file: /cvs/src/usr.bin/banner/banner.1,v
>> retrieving revision 1.8
>> diff -u -p -r1.8 banner.1
>> --- src/usr.bin/banner/banner.1  6 Aug 2007 19:16:06 -   1.8
>> +++ src/usr.bin/banner/banner.1  23 Sep 2015 23:40:36 -
>> @@ -24,7 +24,7 @@
>>  .Nd print strings in large letters
>>  .Sh SYNOPSIS
>>  .Nm banner
>> -.Ar string ...
>> +.Op Ar string ...
>>  .Sh DESCRIPTION
>>  .Nm
>>  prints up to 10 characters of each



Re: banner(1): fix SYNOPSIS

2015-09-24 Thread Jason McIntyre
On Wed, Sep 23, 2015 at 07:42:54PM -0400, Michael Reed wrote:
> banner(1) without arguments works fine, so denote it as such
> in the manual.
> 

in the context of banner(1), define "works fine". i mean, technically
maybe it does but, practically, why try to show this to the reader? is
there a use for banner without arguments?

jmc

> 
> 
> Index: src/usr.bin/banner/banner.1
> ===
> RCS file: /cvs/src/usr.bin/banner/banner.1,v
> retrieving revision 1.8
> diff -u -p -r1.8 banner.1
> --- src/usr.bin/banner/banner.1   6 Aug 2007 19:16:06 -   1.8
> +++ src/usr.bin/banner/banner.1   23 Sep 2015 23:40:36 -
> @@ -24,7 +24,7 @@
>  .Nd print strings in large letters
>  .Sh SYNOPSIS
>  .Nm banner
> -.Ar string ...
> +.Op Ar string ...
>  .Sh DESCRIPTION
>  .Nm
>  prints up to 10 characters of each
> 



banner(1): fix SYNOPSIS

2015-09-23 Thread Michael Reed
banner(1) without arguments works fine, so denote it as such
in the manual.



Index: src/usr.bin/banner/banner.1
===
RCS file: /cvs/src/usr.bin/banner/banner.1,v
retrieving revision 1.8
diff -u -p -r1.8 banner.1
--- src/usr.bin/banner/banner.1 6 Aug 2007 19:16:06 -   1.8
+++ src/usr.bin/banner/banner.1 23 Sep 2015 23:40:36 -
@@ -24,7 +24,7 @@
 .Nd print strings in large letters
 .Sh SYNOPSIS
 .Nm banner
-.Ar string ...
+.Op Ar string ...
 .Sh DESCRIPTION
 .Nm
 prints up to 10 characters of each