Re: PATCH: bugfix for broken AC_CONFIG_xxx in 2.50

[Akim Demaille]

>>>>> "Paolo" == Paolo Bonzini <address@hidden> writes:

Paolo> The command line for config.status is read before the INIT-CMDS
Paolo> are evaluated, so using $xxx in AC_CONFIG_xxx macros won't work
Paolo> (such as in the example for AC_CONFIG_LINKS found in the info
Paolo> documentation).

Sorry, but I don't understand what could be the relationship.  Could
you detail what happens?  Do you have a small test case?

Paolo> The fix is simply to move the INIT-CMDS output phase right
Paolo> after AS_SHELL_SANITIZE.

Running the INIT-CMDS before the handling of options seems wrong to
me, as they are not to be run when --help etc.

Paolo> This bug is very critical, since most packages using
Paolo> AC_CONFIG_LINKS will likely be broken (using $xxx with symbolic
Paolo> links is very common)!  The only workaround (other than patch
Paolo> patch patch) is not to use 2.50!

I have no problems with Gettext which uses it though.