Re: kludgey patch for Ultrix4.4 failure

[Jim Meyering]

Akim Demaille <address@hidden> wrote:
| | This fixes that nagging `illegal io' problem with Ultrix's /bin/sh.
| | (Thanks again for the account, Harlan!)
...
| But given the semantics I was trying to give to all these macros, it

Right.  That's why I called my change a kludge.

| seems to me that the actually fix should be:
|
|   AC_DEFUN([_AC_PREPROC_IFELSE],
|   [m4_ifvaln([$1], [AC_LANG_CONFTEST([$1])])dnl
| - if _AC_EVAL_STDERR([$ac_cpp conftest.$ac_ext >/dev/null]); then
|
| becoming
|
|   AC_DEFUN([_AC_PREPROC_IFELSE],
|   [m4_ifvaln([$1], [AC_LANG_CONFTEST([$1])])dnl
| + if _AC_EVAL_STDERR([$ac_cpp conftest.$ac_ext]) >/dev/null; then
|
| i.e., the /dev/null pulled outside.  This lets AC_EVAL_STDERR keep its
| eval, and given that there is already a () provided by
| _AC_EVAL_STDERR, it should be enough.

That looks great.
Thanks!

I assume you'll check it in.