Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-07 Thread Jason McIntyre
On Fri, Dec 07, 2018 at 07:48:39PM +1100, Ross L Richardson wrote: > > On Fri, Dec 07, 2018 at 06:42:51AM +, Jason McIntyre wrote: > >[...] > > > > morning. > > > > i'm afraid i think this reads quite poorly - it is hard to actually > > understand what is meant. > > > > here's the thing - s

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-07 Thread Ross L Richardson
On Fri, Dec 07, 2018 at 06:42:51AM +, Jason McIntyre wrote: >[...] > > morning. > > i'm afraid i think this reads quite poorly - it is hard to actually > understand what is meant. > > here's the thing - someone has already written the text. if we want to > change the author's text, i think

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Jason McIntyre
On Fri, Dec 07, 2018 at 02:56:03PM +1100, Ross L Richardson wrote: > > Whilst I agree with Otto (and others), how about making it really simple > by avoiding technical terms? > > Ross > > > Index: ifstated.conf.5 > === > RCS fi

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Ross L Richardson
Whilst I agree with Otto (and others), how about making it really simple by avoiding technical terms? Ross Index: ifstated.conf.5 === RCS file: /cvs/src/usr.sbin/ifstated/ifstated.conf.5,v retrieving revision 1.13 diff -u -p -

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Stuart Henderson
On 2018/12/06 13:01, Ted Unangst wrote: > Jason McIntyre wrote: > > On Thu, Dec 06, 2018 at 10:21:47PM +1100, Ross L Richardson wrote: > > > > > > The number is in seconds, but that's currently not specified. > > > > > > Wording which preserved "frequency" but made sense with "seconds" > > > elud

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Jason McIntyre
On Thu, Dec 06, 2018 at 01:01:49PM -0500, Ted Unangst wrote: > Jason McIntyre wrote: > > On Thu, Dec 06, 2018 at 10:21:47PM +1100, Ross L Richardson wrote: > > > > > > The number is in seconds, but that's currently not specified. > > > > > > Wording which preserved "frequency" but made sense with

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Jason McIntyre
On Thu, Dec 06, 2018 at 06:40:28PM +0100, Otto Moerbeek wrote: > On Thu, Dec 06, 2018 at 05:33:21PM +, Jason McIntyre wrote: > > > On Thu, Dec 06, 2018 at 06:26:36PM +0100, Otto Moerbeek wrote: > > > On Thu, Dec 06, 2018 at 05:22:36PM +, Jason McIntyre wrote: > > > > > > > On Thu, Dec 06,

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Ted Unangst
Jason McIntyre wrote: > On Thu, Dec 06, 2018 at 10:21:47PM +1100, Ross L Richardson wrote: > > > > The number is in seconds, but that's currently not specified. > > > > Wording which preserved "frequency" but made sense with "seconds" > > eluded me, so I changed things to refer to "interval". > >

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Otto Moerbeek
On Thu, Dec 06, 2018 at 05:33:21PM +, Jason McIntyre wrote: > On Thu, Dec 06, 2018 at 06:26:36PM +0100, Otto Moerbeek wrote: > > On Thu, Dec 06, 2018 at 05:22:36PM +, Jason McIntyre wrote: > > > > > On Thu, Dec 06, 2018 at 06:17:16PM +0100, Otto Moerbeek wrote: > > > > On Thu, Dec 06, 201

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Jason McIntyre
On Thu, Dec 06, 2018 at 06:26:36PM +0100, Otto Moerbeek wrote: > On Thu, Dec 06, 2018 at 05:22:36PM +, Jason McIntyre wrote: > > > On Thu, Dec 06, 2018 at 06:17:16PM +0100, Otto Moerbeek wrote: > > > On Thu, Dec 06, 2018 at 05:09:23PM +, Jason McIntyre wrote: > > > > > > > On Thu, Dec 06,

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Otto Moerbeek
On Thu, Dec 06, 2018 at 05:22:36PM +, Jason McIntyre wrote: > On Thu, Dec 06, 2018 at 06:17:16PM +0100, Otto Moerbeek wrote: > > On Thu, Dec 06, 2018 at 05:09:23PM +, Jason McIntyre wrote: > > > > > On Thu, Dec 06, 2018 at 10:21:47PM +1100, Ross L Richardson wrote: > > > > > > > > The nu

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Jason McIntyre
On Thu, Dec 06, 2018 at 06:17:16PM +0100, Otto Moerbeek wrote: > On Thu, Dec 06, 2018 at 05:09:23PM +, Jason McIntyre wrote: > > > On Thu, Dec 06, 2018 at 10:21:47PM +1100, Ross L Richardson wrote: > > > > > > The number is in seconds, but that's currently not specified. > > > > > > Wording

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Otto Moerbeek
On Thu, Dec 06, 2018 at 05:09:23PM +, Jason McIntyre wrote: > On Thu, Dec 06, 2018 at 10:21:47PM +1100, Ross L Richardson wrote: > > > > The number is in seconds, but that's currently not specified. > > > > Wording which preserved "frequency" but made sense with "seconds" > > eluded me, so I

Re: [Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Jason McIntyre
On Thu, Dec 06, 2018 at 10:21:47PM +1100, Ross L Richardson wrote: > > The number is in seconds, but that's currently not specified. > > Wording which preserved "frequency" but made sense with "seconds" > eluded me, so I changed things to refer to "interval". > > > Ross > > > > Index: if

[Patch] Specify unit for 'every' in ifstated.conf.5

2018-12-06 Thread Ross L Richardson
The number is in seconds, but that's currently not specified. Wording which preserved "frequency" but made sense with "seconds" eluded me, so I changed things to refer to "interval". Ross Index: ifstated.conf.5 === RCS file