On 08/09/16 18:28, Bruce Korb wrote: > Index: fixincludes/fixincl.tpl > > =================================================================== > > --- fixincludes/fixincl.tpl (revision 239193) > > +++ fixincludes/fixincl.tpl (working copy) > > @@ -1,7 +1,7 @@ > > [= AutoGen5 Template -*- Mode: C -*- > > x=fixincl.x =] > > [= > > - (if (version-compare >= autogen-version "5.18") > > + (if (version-compare >= autogen-version "5.18.1") > > (dne "-D" " * " "/* ") > > (dne " * " "/* ") ) =] > > Was this causing a problem? If so, then it all looks good to me. I > had thought the "-D" was implemented in 5.18, but that was long, long > ago now. >
Yes. I was surprised that genfixes generated a bogus fixincl.x there was no warning, but the result did not compile in the end. You can see the result as the first obsoleted patch in the PR 72833. I use autogen 5.18 from Ubuntu 14.04. Thanks Bernd. > On Tue, Aug 9, 2016 at 5:17 AM, Bernd Edlinger > <bernd.edlin...@hotmail.de> wrote: >> Hi! >> >> Due to my recent cleanup of special_function_p the darwin bootstrap is >> currently broken because the longjmp function misses the noreturn >> attribute. Therefore I revived an old solaris_longjmp_noreturn >> fixinclude rule. >> >> See the attached patch. >> >> Boot-strap on i686-apple-darwin11 and x86_64-apple-darwin11 was >> successful. >> >> Is it OK for trunk? >> >> >> Thanks >> Bernd.