Re: [Gnash-commit] gnash macros/curl.m4 ChangeLog

[strk]

On Sun, Sep 03, 2006 at 12:52:46AM +0000, Rob Savoye wrote:
> CVSROOT:      /sources/gnash
> Module name:  gnash
> Changes by:   Rob Savoye <rsavoye>    06/09/03 00:52:46
> 
> Modified files:
>       macros         : curl.m4 
>       .              : ChangeLog 
> 
> Log message:
>               * configure.ac: Check to see if libCurl supports RTMP.
>               * macros/curl.m4: Add support to use curl-config. Look to see 
> what
>               protocols it supports.
> 

...

>    dnl If the path hasn't been specified, go look for it.
>    if test x"${ac_cv_path_curl_incl}" = x; then
> +    if test x"${curlconfig}" != "x" ; then
> +      ac_cv_path_curl_incl=`${curlconfig} --cflags`
> +    else
> +      if test x"${ac_cv_path_curl_incl}" = x ; then
>      AC_CHECK_HEADERS(curl/curl.h, [ac_cv_path_curl_incl=""],[
>        if test x"${ac_cv_path_curl_incl}" = x; then
>          AC_MSG_CHECKING([for libcurl header])

Rob, this patch broken the code as AC_CHECK_HEADERS is NOT
invoked if curl-config is found. This skips defining HAVE_CURL_CURL_H
while we *need* that definition from within StreamProvider.cpp to
enable use of CURL.

I've fixed a similar issue for gstreamer in the past.
See gstreamer.m4 for details. Basically I was always calling AC_CHECK_HEADERS
after setting CFLAGS and CPPFLAGS appropriately...

--strk;