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 thingy seems fine, too. RE: https://gcc.gnu.org/ml/gcc-patches/2018-06/msg00370.html On 06/09/18 06:22, Rasmus Villemoes wrote: > 2018-06-09 Rasmus Villemoes <rasmus.villem...@prevas.dk> > > fixincludes/ > > * genfixes: exit 1 when autogen not found. > * genfixes: Remove some redundant code. > * genfixes: Update URL to autogen source code. > --- > fixincludes/genfixes | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/fixincludes/genfixes b/fixincludes/genfixes > index f0fc5e64f8b..47aad01289d 100755 > --- a/fixincludes/genfixes > +++ b/fixincludes/genfixes > @@ -55,19 +55,16 @@ do > esac > done > > -if [ $# -eq 0 ] ; then > - set -- fixincl.x > -fi > - > AG="autogen $AG" > set -e > > if [ -z "`${AG} -v | fgrep ' 5.'`" ] > then > echo "AutoGen appears to be out of date or not correctly installed." > - echo "Please download and install:" > - echo " ftp://gcc.gnu.org/pub/gcc/infrastructure/autogen.tar.gz" > + echo "Please download and install from:" > + echo " https://ftp.gnu.org/gnu/autogen/" > touch fixincl.x > + exit 1 > else > echo AutoGen-ing fixincl.x > $AG inclhack.def >