Re: timeout: Prettify man page and usage

2021-09-04 Thread Ingo Schwarze
Hi Jason, Jason McIntyre wrote on Sat, Sep 04, 2021 at 09:47:12PM +0100: > pretty damning that my ok is on that commit ;) > i'll try to remember... Heh. With the amount of work you are doing - your current commit count stands at 9113, on average 1.34 per day, during a time of over eightteen yea

Re: timeout: Prettify man page and usage

2021-09-04 Thread Jason McIntyre
On Sat, Sep 04, 2021 at 02:14:47PM +0200, Ingo Schwarze wrote: > Hi Jason, > > Jason McIntyre wrote on Fri, Sep 03, 2021 at 02:46:47PM +0100: > > On Fri, Sep 03, 2021 at 03:42:21PM +0200, Ingo Schwarze wrote: > >> Theo de Raadt wrote on Thu, Sep 02, 2021 at 09:57:11AM -0600: > > >>> I think we sh

Re: timeout: Prettify man page and usage

2021-09-04 Thread Ingo Schwarze
Hi Jason, Jason McIntyre wrote on Fri, Sep 03, 2021 at 02:46:47PM +0100: > On Fri, Sep 03, 2021 at 03:42:21PM +0200, Ingo Schwarze wrote: >> Theo de Raadt wrote on Thu, Sep 02, 2021 at 09:57:11AM -0600: >>> I think we should list shorts, and longs which have no shorts. >> I agree, and i think we

Re: timeout: Prettify man page and usage

2021-09-03 Thread Jason McIntyre
On Fri, Sep 03, 2021 at 03:42:21PM +0200, Ingo Schwarze wrote: > Hi, > > Theo de Raadt wrote on Thu, Sep 02, 2021 at 09:57:11AM -0600: > > > I think we should list shorts, and longs which have no shorts. > > I agree, and i think we arrived at the same conclusion in the past. > > It applies to b

Re: timeout: Prettify man page and usage

2021-09-03 Thread Ingo Schwarze
Hi, Theo de Raadt wrote on Thu, Sep 02, 2021 at 09:57:11AM -0600: > I think we should list shorts, and longs which have no shorts. I agree, and i think we arrived at the same conclusion in the past. It applies to both usage() and SYNOPSIS, and ideally, both should match, except maybe in very un

Re: timeout: Prettify man page and usage

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 10:41:32PM +0200, Leon Fischer wrote: > > > > yes, fair point. i also dislike excess markup. but i think in the first > > sentence it is not excess, it is explanation. i mean "duration"is marked > > up. > > > > so that first sentence should try to talk about all those arg el

Re: timeout: Prettify man page and usage

2021-09-02 Thread ropers
> Changed to "duration and time may contain a decimal fraction. The value > defaults to seconds unless an unit-specifying suffix is given." a unit, not an unit It's phonetic; it depends on pronunciation, not on the way it's spelled. Hence "a unit" /ˈjuːnɪt/ but "an unreal amount" /ʌnˈrɪəl/. --

Re: timeout: Prettify man page and usage

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 05:47:09PM +0200, Leon Fischer wrote: > > > @@ -34,83 +35,74 @@ > > > .Nd run a command with a time limit > > > .Sh SYNOPSIS > > > .Nm > > > -.Op Fl Fl signal Ar sig | Fl s Ar sig > > > -.Op Fl Fl preserve-status > > > -.Op Fl Fl kill-after Ar time | Fl k Ar time > > > -.

Re: timeout: Prettify man page and usage

2021-09-02 Thread Todd C . Miller
On Thu, 02 Sep 2021 11:05:24 +0200, Martijn van Duren wrote: > There are a few cases where we don't set errno, but do use err(3). OK millert@ - todd

Re: timeout: Prettify man page and usage

2021-09-02 Thread Theo de Raadt
Jason McIntyre wrote: > On Thu, Sep 02, 2021 at 08:56:29AM +, Job Snijders wrote: > > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > > .Ar time > > > > -can be integer or decimal numbers. > > > > +are positive integer or real (decimal) numbers, with an optional > > >

Re: timeout: Prettify man page and usage

2021-09-02 Thread Jason McIntyre
On Thu, Sep 02, 2021 at 08:56:29AM +, Job Snijders wrote: > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > .Ar time > > > -can be integer or decimal numbers. > > > +are positive integer or real (decimal) numbers, with an optional > > > > can you have a negative timeout?

Re: timeout: Prettify man page and usage

2021-09-02 Thread Sebastian Benoit
ok Martijn van Duren(openbsd+t...@list.imperialat.at) on 2021.09.02 11:05:24 +0200: > On Thu, 2021-09-02 at 08:56 +, Job Snijders wrote: > > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > > .Ar time > > > > -can be integer or decimal numbers. > > > > +are positive int

Re: timeout: Prettify man page and usage

2021-09-02 Thread Martijn van Duren
On Thu, 2021-09-02 at 08:56 +, Job Snijders wrote: > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > .Ar time > > > -can be integer or decimal numbers. > > > +are positive integer or real (decimal) numbers, with an optional > > > > can you have a negative timeout? > > N

Re: timeout: Prettify man page and usage

2021-09-02 Thread Stuart Henderson
On 2021/09/02 08:56, Job Snijders wrote: > On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > > .Ar time > > > -can be integer or decimal numbers. > > > +are positive integer or real (decimal) numbers, with an optional > > > > can you have a negative timeout? > > Negative values

Re: timeout: Prettify man page and usage

2021-09-02 Thread Job Snijders
On Thu, Sep 02, 2021 at 07:23:26AM +0100, Jason McIntyre wrote: > > .Ar time > > -can be integer or decimal numbers. > > +are positive integer or real (decimal) numbers, with an optional > > can you have a negative timeout? Negative values are not permitted $ timeout -- -1 /bin/ls timeout: inva

Re: timeout: Prettify man page and usage

2021-09-01 Thread Jason McIntyre
On Wed, Sep 01, 2021 at 09:29:35PM +0200, Leon Fischer wrote: > Here's my thanks for importing timeout(1). > > P.S. The wording could still be improved, especially the -k description. > hi. > Index: timeout.1 > === > RCS file: /cvs