Re: [PATCH 04/12] use a shell function for _AC_RUN_IFELSE

[Ralf Wildenhues]

Hi Eric,

* Eric Blake wrote on Thu, Oct 23, 2008 at 02:37:22PM CEST:
> +2008-10-23  Paolo Bonzini  <address@hidden>
> +     and Eric Blake  <address@hidden>
> +
> +     Use a shell function for _AC_RUN_IFELSE.
> +        * lib/autoconf/general.m4 (_AC_RUN_IFELSE_BODY): New macro.
> +     (_AC_RUN_IFELSE): Use a shell function.
> +     (_AC_RUN_LOG): Avoid subshell for logging.
> +

> --- a/lib/autoconf/general.m4
> +++ b/lib/autoconf/general.m4
> @@ -2230,7 +2230,7 @@ AU_ALIAS([AC_VERBOSE], [AC_MSG_RESULT])
>  # ----------------------------------
>  # Eval COMMAND, save the exit status in ac_status, and log it.
>  AC_DEFUN([_AC_RUN_LOG],
> -[{ ($2) >&AS_MESSAGE_LOG_FD
> +[{ { $2; } >&AS_MESSAGE_LOG_FD

This breaks Ultrix compatibility (when used as part of AC_TRY_EVAL), no?

>    ($1) 2>&AS_MESSAGE_LOG_FD
>    ac_status=$?
>    _AS_ECHO_LOG([\$? = $ac_status])

Cheers,
Ralf