ID: 50303 User updated by: galtgendo at o2 dot pl Reported By: galtgendo at o2 dot pl Status: Bogus Bug Type: Compile Failure Operating System: Linux PHP Version: 5.2.11 New Comment:
As for pkg-config, it's just an idea, but: - it's cleaner - what if curl does the same thing that gnutls did - drops it's custom script ? Previous Comments: ------------------------------------------------------------------------ [2009-11-26 11:07:28] galtgendo at o2 dot pl As for 1 - I could quote a part of autoconf info page at you once again, but ...oh,well As for 2 - in ext/curl/interface.c you have: # if defined(HAVE_GCRYPT_H) # define PHP_CURL_NEED_GNUTLS_TSL # include <gcrypt.h> # else ... # endif But if you use AC_CHECK_HEADER, HAVE_GCRYPT_H never gets defined, even if the header gets properly detected. And I don't see how any of AC_TRY_RUN in ext/curl/config.m4 could succeed without libcurl being added to LIBS for those tests. ------------------------------------------------------------------------ [2009-11-26 10:05:46] j...@php.net 1. Wont fix 2. No idea what bug that is. And last but not least: DO NOT report multiple issues in one report. ------------------------------------------------------------------------ [2009-11-26 00:35:42] galtgendo at o2 dot pl Description: ------------ Seeing that bug 50291 got accepted, I'd like to readdress a few issues, some of which I already address here (some of them were bogus, other were not). 1. Gentoo bug 151444 - '--as-needed' problem a.k.a. using LDFLAGS instead of libs while Gentoo is using a different fix now, I still keep a patch (quite a bit reduced, compared to the one in the bug), that modifies PHP_CHECK_LIBRARY to use LIBS instead of LDFLAGS - it modifies a few other ext specific macros too; it "works for me" for over a year now, but I'm not building all exts, so it could use some polishing 2. Gentoo bug 275240 - curl.m4 while changing from curl-config to pkg-config is just an idea (curl, just as php needs to be kept up to date, due to security, so it shouldn't be a big problem), AC_CHECK_HEADER -> AC_CHECK_HEADERS is required for things to work ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=50303&edit=1
