ID:               38844
 Updated by:       [EMAIL PROTECTED]
 Reported By:      wirth at bison-soft dot de
-Status:           Open
+Status:           Closed
 Bug Type:         Compile Failure
 Operating System: linux 2.4 (RHEL 3)
 PHP Version:      5.2.0RC4
 New Comment:

Fixed in CVS, thanks.


Previous Comments:
------------------------------------------------------------------------

[2006-09-15 15:35:15] wirth at bison-soft dot de

the curl_easy_strerror is gone, but there is also exactly
one occurence of curl_multi_strerror (also introduced in 7.12.0) which
leads to another fail. I've commented out
the concerning line 438 in my local source tree and than the build
finished proberly.

Btw., do you think it's wise to simply log nothing when the
curl_*_strerror type functions are not available? I think something
like

#if HAVE_CURL_EASY_STRERROR
    php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s",   
curl_easy_strerror(msg->data.result));
#else
    php_error_docref(NULL TSRMLS_CC, E_WARNING, "There was an error
code=%d", msg->data.result);
#endif

would be more convenient for the user, and same for the
curl_multi_strerror line in streams.c (assuming there
is already an HAVE_CURL_MULTI_STRERROR test added):

#if HAVE_CURL_MULTI_STRERROR
    php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s",
curl_multi_strerror(m));
#else
    php_error_docref(NULL TSRMLS_CC, E_WARNING, "There was an error
mcode=%d", m);
#endif

------------------------------------------------------------------------

[2006-09-15 13:44:58] [EMAIL PROTECTED]

This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.



------------------------------------------------------------------------

[2006-09-15 13:27:00] wirth at bison-soft dot de

Description:
------------
Compile breaks at linkage stage with

ext/curl/streams.o(.text+0xacf): In function `php_curl_stream_opener':
/usr/src/redhat/BUILD/php-5.2.0RC4/ext/curl/streams.c:448: undefined
reference to `curl_easy_strerror'

curl_easy_strerror is introduced in libcurl 7.12.0,
RHEL3 includes only libcurl 7.10.6 and your documentation
states "PHP 5.0.0 requires a libcurl version 7.10.5 or greater".

Please either upgrade your documentation or (preferred)
get rid of curl_easy_strerror which only occurs on one
place (streams.c line 448) which would your users free
from the hassel to upgrade curl especially for a new
php version :-).




------------------------------------------------------------------------


-- 
Edit this bug report at http://bugs.php.net/?id=38844&edit=1

Reply via email to