Re: AC_PROG_MAKE_SET problem, patch included (fwd)

[Dan McMahill]

On 22 Sep 2001, Akim Demaille wrote:

> >>>>> "mcmahill" == mcmahill  <address@hidden> writes:
> 
> mcmahill> address@hidden 493 # autoconf --version Autoconf version 2.13
> 
> Sorry, but your problem in incomprehensible, and 2.13 is obsolete now.
> 

Let me be more specific.  Also the problem exists in 2.52 as well.


Original message:

>when using the MINGW compilers under cygwin, the AC_PROG_MAKE_SET macro
>fails because you get an unterminated quote coming out of one of the
>tests.  For some reason, the first " below was getting eaten.  Adding the
>space seems to fix it.
>
>-Dan
>
>
>address@hidden 492 # diff -U2 acspecific.m4.orig acspecific.m4
>--- acspecific.m4.orig  Tue Sep 18 16:01:37 2001
>+++ acspecific.m4       Tue Sep 18 16:02:08 2001
>@@ -455,5 +455,5 @@
> [cat > conftestmake <<\EOF
> all:
>-       @echo 'ac_maketemp="${MAKE}"'
>+       @echo 'ac_maketemp=" ${MAKE} "'
> EOF
> changequote(, )dnl
>
>address@hidden 493 # autoconf --version
>Autoconf version 2.13


more details:

The symptom:

checking build system type... i586-pc-cygwin
checking host system type... i586-pc-cygwin
checking for gcc... gcc
[snip]
checking whether make sets ${MAKE}... eval: 1: Syntax error: Unterminated 
quoted string


More info:

The code above which is patched, creates a file 'conftestmake' that looks
like the following.

all:
        @echo 'ac_maketemp="${MAKE}"'


now, I try 2 cases:

% /usr/bin/make -f conftest.make
ac_maketemp="/usr/bin/make"

% /cygdrive/c/mingw/bin/make -f conftest.make
ac_maketemp=:/mingw/bin/make.exe"

Note the missing leading ".  Since the configure script tries to do:
  
  eval `${MAKE-make} -f conftestmake 2>/dev/null | grep temp=`

we get an error.  Now, if (and I don't know why this does it) I add the
extra spaces as shown in the patch, I get:

% /usr/bin/make -f conftest.make
ac_maketemp=" /usr/bin/make "

and

% /cygdrive/c/mingw/bin/make -f conftest.make
ac_maketemp=" c:/mingw/bin/make.exe "

and the configure script then runs.


Hope this clarifies things.

-Dan