2008/5/7, cvs_at_labb.contactor.se wrote:
> Modified Files:
> CHANGES configure.ac
> Log Message:
> - Doug Goldstein provided a configure patch: updates autoconf 2.13 usage to
> autoconf 2.57 usage (which is the version you have specified as the minimum
> version). It's a minor change but it does clean up some warnings with newer
> autoconf (specifically 2.62).
>
> [...]
>
> AC_AIX
> AC_PROG_CC
> +AC_PROG_CC_STDC
> +AM_PROG_CC_C_O
> AC_PROG_INSTALL
>
> [...]
This addition of AC_PROG_CC_STDC and AM_PROG_CC_C_O (or this whole
patch) seem to have caused failure of one of curl's daily autobuilds.
Specifically the one that failed is "Linux 2.6 x86-64 icc 10.1"
http://curl.haxx.se/auto/log.cgi?id=20080508012627-10983 attached log
file build-20080508.log
That same autobuild succeeded the day before the inclusion of this
patch http://curl.haxx.se/auto/log.cgi?id=20080507013806-15127
attached log file build-20080507.log
Some things that can easily be observed with the patch applied are:
1) icc emits following warning: icc: command line remark #10010:
option '-c99' is deprecated and will be removed in a future release.
See '-help deprecated'
2) autoconf's configure script fails to detect the presence of
arpa/nameser.h and arpa/nameser_compat.h header files, causing the
build process to fail.
It would be interesting to know why do we really need AC_PROG_CC_STDC
and AM_PROG_CC_C_O for c-ares library. After all, c-ares library does
not link twice the same source code file compiled with different
options.
I'm noy saying that the patch should be removed. I'm only saying that
some further investigation should be done on this.
I cannot fine Doug Goldstein's report. Maybe the warnings he says that
autoconf 2.62 emmits could or should be addressed somehow else.
googling for the before mentioned macros gives some scaring results.
-- -=[Yang]=-