Re: dhclient.conf(5): manual tweaks

2017-07-30 Thread Jason McIntyre
On Mon, Jul 31, 2017 at 08:34:02AM +0200, Anton Lindqvist wrote: > Hi, > Some suggestions on how to improve the dhclient.conf(5) manual: > > - Shorten initial description, inspired by httpd.conf(5). > > - No need to mention the choice of parser implementation (recursive > descent). > > - Do no

dhclient.conf(5): manual tweaks

2017-07-30 Thread Anton Lindqvist
Hi, Some suggestions on how to improve the dhclient.conf(5) manual: - Shorten initial description, inspired by httpd.conf(5). - No need to mention the choice of parser implementation (recursive descent). - Do not use the word `end` twice in the sentence on comments, instead borrow a sentence

Re: [patch] httpd: don't add date header if already set

2017-07-30 Thread Florian Obser
On Sun, Jul 30, 2017 at 09:43:09PM -0400, Ted Unangst wrote: > Florian Obser wrote: > > OK florian@ > > > > p.s. whoever commits this please add extra ( ) around the && term, > > Apparently clang no longer supports operator precedence *sigh*. > > > > I'm wondering if it also warns for a * b + c >

Re: [patch] httpd: don't add date header if already set

2017-07-30 Thread Ted Unangst
Florian Obser wrote: > OK florian@ > > p.s. whoever commits this please add extra ( ) around the && term, > Apparently clang no longer supports operator precedence *sigh*. > > I'm wondering if it also warns for a * b + c > > /usr/src/usr.sbin/httpd/server_fcgi.c:665:56: warning: '&&' within '||'

Re: efiboot boot path matching

2017-07-30 Thread Mark Kettenis
> Date: Sun, 30 Jul 2017 12:20:27 +1000 > From: Jonathan Gray > > On Sat, Jul 29, 2017 at 09:01:35PM +0200, Patrick Wildt wrote: > > On Sat, Jul 29, 2017 at 02:59:19PM +0200, Mark Kettenis wrote: > > > This is apparently very hard. Caught this on arm64 where > > > efi_device_path_depth() returne

[patch/bgpd] Move comment in rde.c

2017-07-30 Thread Denis Fondras
Hi, A comment is misplaced in rde.c. Index: rde.c === RCS file: /cvs/src/usr.sbin/bgpd/rde.c,v retrieving revision 1.368 diff -u -p -r1.368 rde.c --- rde.c 29 May 2017 13:10:40 - 1.368 +++ rde.c 30 Jul 2017 19:49

Re: make(1): minor manual tweaks

2017-07-30 Thread Jason McIntyre
On Sun, Jul 30, 2017 at 08:26:37PM +0200, Anton Lindqvist wrote: > Hi, > I count six operators so instead stop mentioning the total number of > operators. > i agree. we need to stop trying to list how many things we currently support: it's a recipe for madness. i've added a comment inline though.

make(1): minor manual tweaks

2017-07-30 Thread Anton Lindqvist
Hi, I count six operators so instead stop mentioning the total number of operators. While here, add a missing `Pp` to make the display block consistent with the one above. Comments? OK? Index: make.1 === RCS file: /cvs/src/usr.bin/m

Re: efiboot boot path matching

2017-07-30 Thread YASUOKA Masahiko
Hi, On Sun, 30 Jul 2017 12:20:27 +1000 Jonathan Gray wrote: > On Sat, Jul 29, 2017 at 09:01:35PM +0200, Patrick Wildt wrote: >> On Sat, Jul 29, 2017 at 02:59:19PM +0200, Mark Kettenis wrote: >> > This is apparently very hard. Caught this on arm64 where >> > efi_device_path_depth() returned 0, wh

Re: sysctl machdep.lidaction=suspend

2017-07-30 Thread Marco Bonetti
On 07/29, Martin Natano wrote: > Words are more descriptive than numbers. Let's use them! > > This diff removes > > sysctl machdep.lidaction=0 > sysctl machdep.lidaction=1 > sysctl machdep.lidaction=2 > > in favor of the more descriptive > > sysctl machdep.lidaction=none

Re: Add machdep.lidaction=3 - powerdown laptop upon lid closing

2017-07-30 Thread Marco Bonetti
On 07/27, Bryan Steele wrote: > On Fri, Jul 28, 2017 at 12:09:17AM +0100, Marco Bonetti wrote: > > It looks like spaces and tabs got mangled, I'm resending the diff in-line in > > this email. > > > > Any other comments / OKs ? > > > > Index: etc/etc.amd64/sysctl.conf > > =

Re: [PATCH] run security(8) on first boot

2017-07-30 Thread Robert Peichaer
On Sat, Jul 29, 2017 at 05:25:51PM -0400, Joe Gidi wrote: > I did a couple of fresh installs the other day, which reminded me of a > minor irritation and prompted me to think about a possible solution. > > The first run of security(8) on a fresh install is not terribly helpful. > It produces a hug

disable DEBUG code from libunwind (part of libcxxabi)

2017-07-30 Thread Sebastien Marie
Hi, Digging into unwinding code, I found that per default some DEBUG stuff is enable (NDEBUG preprocessor macro isn't defined). The debug code is logging/tracing feature activated on environment variable presence (without any issetugid(2) check) ; and some uncondionnal logging on failure for ::pt

Re: less(1) - segmentation fault with '-g'

2017-07-30 Thread Anton Lindqvist
Any OKs for the diff below? On Fri, Jul 28, 2017 at 01:32:34PM +0200, Anton Lindqvist wrote: > On Fri, Jul 28, 2017 at 01:19:39PM +0200, Theo Buehler wrote: > > On Thu, Jul 27, 2017 at 10:31:51AM +0100, Larry Hynes wrote: > > > Hi > > > > > > $ env | grep LESS > > > LESSHISTFILE=- > > > LESS="-i

Re: [diff] httpd: tls client cert & CRL checks

2017-07-30 Thread Jack Burton
On Sat, 29 Jul 2017 20:49:18 +0200 Jan Klemkow wrote: > On Fri, Jul 28, 2017 at 02:05:34AM +0930, Jack Burton wrote: > > On Thu, 27 Jul 2017 13:10:14 +0200 > > > > > But, I found a bug in the part of the FastCGI variables. The > > > following condition is always false. > > > > > > > Index:

Re: mg: print default choice before colon in echo line prompts

2017-07-30 Thread Florian Obser
OK florian@ On Sat, Jul 29, 2017 at 04:23:24PM -0500, Scott Cheloha wrote: > Hi, > > GNU Emacs always, or very nearly always, prints the default > choice before the colon in any given prompt. For example, > you'll see > > "Kill buffer (default foo): " > > not > > "Kill buffer: (de