]> granicus.if.org Git - curl/commit
configure: ac_cv_ -> curl_cv_ for write-only vars
authorIrfan Adilovic <i.adilovic@nfotex.com>
Sun, 17 Apr 2016 20:05:15 +0000 (22:05 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Thu, 21 Apr 2016 21:08:28 +0000 (23:08 +0200)
commitd9f3b365a3b663d6e45ff734a86b313e2fbcbbf2
tree18cc55e54d13894a1ad6daf8e0825f7b86bbb0f1
parent4b639dbc747e69b2e017d1e1b8b2a9beb4711bfd
configure: ac_cv_ -> curl_cv_ for write-only vars

These configure vars are modified in a curl-specific way but never
evaluated or loaded from cache, even though they are designated as
_cv_. We could either implement proper AC_CACHE_CHECKs for them, or
remove them completely.

Fixes #603 as ac_cv_func_gethostbyname is no longer clobbered, and
AC_CHECK_FUNC(gethostbyname...) will no longer spuriously succeed after
the first configure run with caching.

`ac_cv_func_strcasecmp` is curious, see #770.

`eval "ac_cv_func_$func=yes"` can still cause problems as it works in
tandem with AC_CHECK_FUNCS and then potentially modifies its result. It
would be best to rewrite this test to use a new CURL_CHECK_FUNCS macro,
which works the same as AC_CHECK_FUNCS but relies on caching the values
of curl_cv_func_* variables, without modifiying ac_cv_func_*.
acinclude.m4
m4/curl-functions.m4