Re: [PATCH] fixincludes: a few genfixes changes

2018-06-12 Thread Jeff Law
On 06/12/2018 08:25 AM, Bruce Korb wrote: > This looks pretty reasonable to me. I'm sure "set -- fixincl.x" > meant something two decades ago when I wrote it. :) > I should update my e-address since I can be off gnu mail for > weeks at a time nowadays. (We retired folk are less consistent > about

Re: [PATCH] fixincludes: a few genfixes changes

2018-06-12 Thread Bruce Korb
This looks pretty reasonable to me. I'm sure "set -- fixincl.x" meant something two decades ago when I wrote it. :) I should update my e-address since I can be off gnu mail for weeks at a time nowadays. (We retired folk are less consistent about checking project emails ...) Oh, the assert.h

[PATCH] fixincludes: a few genfixes changes

2018-06-09 Thread Rasmus Villemoes
It's useful to have genfixes fail when autogen is not found, especially when genfixes is run as part of an automated build. That's also more consistent with what happens if autogen is found, but the run fails for some reason (set -e is in effect). Setting $@ to fixincl.x when $# is 0 doesn't seem