autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

status.m4: $INSTALL and quoting


From: Ralf Wildenhues
Subject: status.m4: $INSTALL and quoting
Date: Sun, 19 Mar 2006 15:06:13 +0100
User-agent: Mutt/1.5.9i

This Autoconf patch:

| 2005-07-25  Stepan Kasal  <address@hidden>
| 
|         Rewrite substantial part of lib/autoconf/status.m4.
|         The main change is that CONFIG_FILES, CONFIG_HEADERS, CONFIG_LINKS,
|         and CONFIG_COMMANDS are not processed in four separate loops.
|         Instead, there is one main loop.  This alows that the common code
|         is expanded only once, thus config.status (and configure) is smaller.

changes CVS Autoconf lib/autoconf/status.m4:_AC_OUTPUT_CONFIG_STATUS
which has:

| AC_PROVIDE_IFELSE([AC_PROG_INSTALL],
| [INSTALL='$INSTALL'
| ])

and had before:
| AC_PROVIDE_IFELSE([AC_PROG_INSTALL],
| [dnl Leave those double quotes here: this $INSTALL is evaluated in a
| dnl here document, which might result in `INSTALL=/bin/install -c'.
| INSTALL="$INSTALL"

If now the previous value of `$INSTALL' is picked up by an old
config.cache file, config.status ends up containing:

| INSTALL=''/usr/bin/install -c''

and throwing some "interesting" errors.  After removing the config.cache
things seem to work again:

| INSTALL='/usr/bin/install -c'

but I have not yet found the corresponding quoting change that
compensates this.  I was also unable to find a reason for the change
in quotation.


The gist of this is that I think that users will have to throw away
their config.cache for every project transitioning from 2.59 to 2.60
due to this change.

(Just in case you're interested, I have config.cache files lying around
that are several years old.)

Cheers,
Ralf




reply via email to

[Prev in Thread] Current Thread [Next in Thread]