Re: autoconf.ac

I have finally had a change to look at Ben's bug report concerning autoconf.ac. Ben> This chunk at the end looks wrong: Ben> ＠＠ -5660,10 +5675,8 ＠＠ Ben> dnl Create a GNUmakefile and Makefile from Makefile.in. Ben> changequote(<<,>>)dnl Ben> -dnl CPP_MAKEFILE(CPPFLAGS,filename) Ben> -define(<<CPP_MAKEFILE>>, Ben> -echo creating $dir/<<$2>> Ben> -$CPP -I. -I${top_srcdir}/src <<$1>> junk.c \ Ben> +echo creating $dir/Makefile Ben> +$CPP -I. -I${top_srcdir}/src -DUSE_GNU_MAKE junk.c \ Ben> dnl Delete line directives inserted by $CPP Ben> | sed -e 's/^\#.*//' \ Ben> dnl Delete spurious blanks inserted by $CPP Ben> ＠＠ -5678,15 +5691,8 ＠＠ Ben> s/\"[ TAB]*$// Ben> }' > Makefile.new Ben> chmod 444 Makefile.new Ben> - mv -f Makefile.new <<$2>> Ben> -)dnl CPP_MAKEFILE Ben> - Ben> - CPP_MAKEFILE(,Makefile) Ben> - CPP_MAKEFILE(-DUSE_GNU_MAKE,GNUmakefile) Ben> - if test -r "xemacs.def.in"; then Ben> - cp -f xemacs.def.in junk.c Ben> - CPP_MAKEFILE(,xemacs.def) Ben> - fi Ben> + mv -f Makefile.new Makefile; cp Makefile GNUmakefile Ben> +dnl Ben> changequote([,])dnl Ben> rm -f junk.c Ben> ) Ben> It deletes the stuff that creates GNUmakefile separately, and just copies Ben> one to the other. I'm mystified by this chunk because it is not in my original patch nor in CVS. I did have it in my workspace for a while because I thought I could eliminate GNUmakefile but I can remember publishing it. In summary, the existing configure.in behaviour is in configure.ac. Malcolm - Going Mad. -- Malcolm Purvis <malcolmp(a)xemacs.org&gt;