[Top][All Lists]
[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
- status.m4: $INSTALL and quoting,
Ralf Wildenhues <=