lock(1): print secondary header only once on SIGINT, SIGQUIT

Hi, If you SIGINT or SIGQUIT at a lock(1) prompt, the secondary header is printed twice, like this: Key: lock: type in the unlock key. timeout in 14:54 minutes lock: type in the unlock key. timeout in 14:54 minutes This is because we call the signal handler that prints

ssl.8: small typo in last commit about subjectAltName

Fix small typo in the last commit of ssl(8): diff --git a/share/man/man8/ssl.8 b/share/man/man8/ssl.8 index 2a7641b921c..70c39e32767 100644 --- a/share/man/man8/ssl.8 +++ b/share/man/man8/ssl.8 @@ -95,7 +95,7 @@ You can also sign the key yourself, using the command: .Ed .Pp Note that some new

grdc: fprintf(stderr -> warnx

Just use warnx. -- Scott Cheloha Index: games/grdc/grdc.c === RCS file: /cvs/src/games/grdc/grdc.c,v retrieving revision 1.26 diff -u -p -r1.26 grdc.c --- games/grdc/grdc.c 7 Mar 2016 12:07:56 - 1.26 +++

Remove getopt from vipw

As far as I can tell the only thing gained from using getopt is handling vipw -- as vipw takes no flags or arguments, is not intended for non-interactive use, and is not POSIX, I don't see a reason -- should be handled. If anyone prefers proper handling of -- perhaps if (!( argc == 1

Re: ping: Style fixes/cleanups

On Tue, Jul 04, 2017 at 11:24:46PM +0200, Klemens Nanni wrote: > On Tue, Jul 04, 2017 at 03:58:13PM +, Florian Obser wrote: > >I like most (all? of the correct ones), can you have another go > >at trying to fix the pointed out changes in behaviour? > > Thanks natano for pointing out these

acme-client: revoke-proc tiny typo

Tiny typo in a comment in acme-client revokeproc.c diff --git a/usr.sbin/acme-client/revokeproc.c b/usr.sbin/acme-client/revokeproc.c index 66ccfa315a2..994ff768675 100644 --- a/usr.sbin/acme-client/revokeproc.c +++ b/usr.sbin/acme-client/revokeproc.c @@ -164,7 +164,7 @@ revokeproc(int fd, const

Re: grdc: fprintf(stderr -> warnx

On Sat, Jul 08, 2017 at 01:56:09PM -0500, Scott Cheloha wrote: > > On Jul 8, 2017, at 1:13 PM, Theo Buehler wrote: > > > > On Sat, Jul 08, 2017 at 10:27:43AM -0500, Scott Cheloha wrote: > >> Just use warnx. > > > > I don't have a real opinion on this one, but note that this

Re: grdc: fprintf(stderr -> warnx

On Sat, Jul 08, 2017 at 10:27:43AM -0500, Scott Cheloha wrote: > Just use warnx. I don't have a real opinion on this one, but note that this changes the output slightly. Moreover, there's another fprintf(stderr, ...) that you might want to convert > > -- > Scott Cheloha > > Index:

lock(1): remove some ARGSUSEDs

style(9) says it's okay to delete these. -- Scott Cheloha Index: usr.bin/lock/lock.c === RCS file: /cvs/src/usr.bin/lock/lock.c,v retrieving revision 1.34 diff -u -p -r1.34 lock.c --- usr.bin/lock/lock.c 3 May 2017 09:51:39 -

Re: grdc: fprintf(stderr -> warnx

> On Jul 8, 2017, at 1:13 PM, Theo Buehler wrote: > > On Sat, Jul 08, 2017 at 10:27:43AM -0500, Scott Cheloha wrote: >> Just use warnx. > > I don't have a real opinion on this one, but note that this changes > the output slightly. Are you worried this will break something?

Re: acme-client: revoke-proc tiny typo

On Sat, Jul 08, 2017 at 02:21:53PM +0200, Hiltjo Posthuma wrote: > Tiny typo in a comment in acme-client revokeproc.c Fixed, thanks.

Re: grdc: fprintf(stderr -> warnx

On Sat, Jul 08, 2017 at 01:56:09PM -0500, Scott Cheloha wrote: > > On Jul 8, 2017, at 1:13 PM, Theo Buehler wrote: > > > > On Sat, Jul 08, 2017 at 10:27:43AM -0500, Scott Cheloha wrote: > >> Just use warnx. > > > > I don't have a real opinion on this one, but note that this

Remove games/fortune/tools

As far as I can tell, it's never been linked into the build even looking in the NetBSD history. unstr is also not linked, but as it's the reverse of strfile (which is built and used but not installed) unstr seems less useless. - Matthew Martin diff --git tools/Do_spell tools/Do_spell deleted