Re: remove needless #ifdef

2020-02-13 Thread Claudio Jeker
On Thu, Feb 13, 2020 at 11:50:46PM +0100, Jan Stary wrote: > On Feb 10 09:28:38, yasu...@openbsd.org wrote: > > Hi, > > > > On Sun, 09 Feb 2020 19:28:50 +0100 > > Jeremie Courreges-Anglas wrote: > > > On Sun, Feb 09 2020, Jan Stary wrote: > > >> Currently, sys/net/pipex_local.h asks #ifdef

Re: dd(1) wording and style

2020-02-13 Thread Jason McIntyre
On Thu, Feb 13, 2020 at 11:25:07PM +0100, Jan Stary wrote: > This diff changes the dd(1) manpage in the following ways: > morning. > * Replace "It Cm if= Ns Ar file" with "It Cm if Ns = Ns Ar file" > and similarly for others. The operand is "if", not "if="; > the "Ns = Ns" might be a

Re: extern already declared

2020-02-13 Thread Todd C . Miller
On Thu, 13 Feb 2020 23:53:49 +0100, Jan Stary wrote: > On Feb 09 09:49:35, mill...@openbsd.org wrote: > > On Sun, 09 Feb 2020 17:46:51 +0100, Jan Stary wrote: > > > > > Whenever unistd.h declares getopt(3), it also declares > > > the extern optind and optarg, so files including unistd.h > > >

Re: extern already declared

2020-02-13 Thread Jan Stary
On Feb 09 17:46:51, h...@stare.cz wrote: > Whenever unistd.h declares getopt(3), it also declares > the extern optind and optarg, so files including unistd.h > don't need to declare those themselves, right? > > Jan > > Index: games/fortune/strfile/strfile.c >

Re: remove needless #ifdef

2020-02-13 Thread Jan Stary
On Feb 10 09:28:38, yasu...@openbsd.org wrote: > Hi, > > On Sun, 09 Feb 2020 19:28:50 +0100 > Jeremie Courreges-Anglas wrote: > > On Sun, Feb 09 2020, Jan Stary wrote: > >> Currently, sys/net/pipex_local.h asks #ifdef __OpenBSD__ > >> and if so, defines "Static" to be nothing, to use it later.

Re: dd(1) wording and style

2020-02-13 Thread Jan Stary
On Feb 13 23:25:07, h...@stare.cz wrote: > This diff changes the dd(1) manpage in the following ways: > > * Replace "It Cm if= Ns Ar file" with "It Cm if Ns = Ns Ar file" > and similarly for others. The operand is "if", not "if="; > the "Ns = Ns" might be a slightly excessive markup, > but

dd(1) wording and style

2020-02-13 Thread Jan Stary
This diff changes the dd(1) manpage in the following ways: * Replace "It Cm if= Ns Ar file" with "It Cm if Ns = Ns Ar file" and similarly for others. The operand is "if", not "if="; the "Ns = Ns" might be a slightly excessive markup, but common: grep -Fr 'Ns = Ns' /usr/share/man | wc -l

vmctl.8, vm.conf.5: DHCP is configured on the first interface only

2020-02-13 Thread Klemens Nanni
VMs may have multiple interface, but only "the [first]" interface gets DHCP if `-L'/`local' is specified. OK? Index: usr.sbin/vmctl/vmctl.8 === RCS file: /cvs/src/usr.sbin/vmctl/vmctl.8,v retrieving revision 1.70 diff -u -p -r1.70

Re: const*ify cfattach

2020-02-13 Thread Theo de Raadt
Visa Hankala wrote: > On Thu, Feb 13, 2020 at 12:00:35PM +0100, Martin Pieuchot wrote: > > These structures are only used by autoconf(9) and don't need to be > > modified. Some subsystems already define most of them as 'const'. > > Diff below turn all the remaining one as such. > > > > Only a

Re: const*ify cfattach

2020-02-13 Thread Visa Hankala
On Thu, Feb 13, 2020 at 06:07:01PM +0100, Martin Pieuchot wrote: > On 13/02/20(Thu) 16:53, Visa Hankala wrote: > > On Thu, Feb 13, 2020 at 12:00:35PM +0100, Martin Pieuchot wrote: > > > These structures are only used by autoconf(9) and don't need to be > > > modified. Some subsystems already

Re: const*ify cfattach

2020-02-13 Thread Martin Pieuchot
On 13/02/20(Thu) 16:53, Visa Hankala wrote: > On Thu, Feb 13, 2020 at 12:00:35PM +0100, Martin Pieuchot wrote: > > These structures are only used by autoconf(9) and don't need to be > > modified. Some subsystems already define most of them as 'const'. > > Diff below turn all the remaining one as

Re: const*ify cfattach

2020-02-13 Thread Visa Hankala
On Thu, Feb 13, 2020 at 12:00:35PM +0100, Martin Pieuchot wrote: > These structures are only used by autoconf(9) and don't need to be > modified. Some subsystems already define most of them as 'const'. > Diff below turn all the remaining one as such. > > Only a single driver, de(4), needed a

Re: apmd: fix autoaction timeout

2020-02-13 Thread Jeremie Courreges-Anglas
On Thu, Feb 13 2020, Jeremie Courreges-Anglas wrote: [...] > - documents the 60 seconds grace period in the manpage That part was not accurate. Updated wording: "After a resume, the effect of those options is inhibited for 60 seconds." Index: apmd.c

Re: apmd: fix autoaction timeout

2020-02-13 Thread Jeremie Courreges-Anglas
On Wed, Feb 12 2020, Scott Cheloha wrote: > On Wed, Feb 12, 2020 at 01:35:22PM +0100, Jeremie Courreges-Anglas wrote: >> On Wed, Feb 12 2020, Jeremie Courreges-Anglas wrote: >> > On Sat, Jan 25 2020, Jeremie Courreges-Anglas wrote: >> >> The diff below improves the way apmd -z/-Z may trigger.