php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #12597 AC_CHECK_LIB on -curl fails if cURL is elsewhere than /usr or /usr/local
Submitted: 2001-08-06 09:58 UTC Modified: 2001-08-06 11:50 UTC
From: heko at iki dot fi Assigned:
Status: Closed Package: *Configuration Issues
PHP Version: 4.0CVS-2001-08-06 OS: OpenBSD 2.9-current
Private report: No CVE-ID: None
View Add Comment Developer Edit
 [2001-08-06 09:58 UTC] heko at iki dot fi 
AC_CHECK_LIB in ext/curl/config.m4 fails if cURL is
installed elsewhere than /usr/local or /usr, regardless
of --with-curl=[directory] setting. The CHECK_LIB macro
provides an optional fourth argument which should in this
case be set to $CURL_LIBS to make sure the test linking
goes ok.

Patch at:

http://saitti.net/~heko/php/patches/patch-ext_curl_config_m4

Or, inline (let's hope it wraps ok):

Index: ext/curl/config.m4
===================================================================
RCS file: /repository/php4/ext/curl/config.m4,v
retrieving revision 1.7
diff -I'$Id' -u -b -B -p -r1.7 config.m4
--- ext/curl/config.m4  27 May 2001 18:52:33 -0000      1.7
+++ ext/curl/config.m4  6 Aug 2001 13:38:38 -0000
@@ -48,7 +48,7 @@ if test "$PHP_CURL" != "no"; then
     AC_DEFINE(HAVE_CURL,1,[ ])
   ],[
     AC_MSG_ERROR(There is something wrong. Please check config.log for more information.)
-  ])
+  ], $CURL_LIBS)
 
   PHP_EXTENSION(curl, $ext_shared)
   PHP_SUBST(CURL_SHARED_LIBADD)

-- 
<---------------------------------------------------------------------->
          Heikki Korpela -- heko@iki.fi -- http://iki.fi/heko/

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2001-08-06 11:37 UTC] sniper@php.net 
Fixed in CVS. You also need to have -L$CURL_DIR/lib after
the $CURL_LIBS since the path is not necessarily given
by curl-config.

--Jani
 [2001-08-06 11:50 UTC] heko at iki dot fi 
Actually, I notified the cURL development list about this
issue, and Daniel Stenberg agreed that -L@libdir@ should
go into curl-config.in.

	http://curl.haxx.se/mail/lib-2001-08/0041.html

Thus, hopefully, -L${CURL_LIB}/lib is not needed after
some future release of cURL.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu May 02 12:01:29 2024 UTC