ID:               49808
 Updated by:       srina...@php.net
 Reported By:      sriram dot natarajan at gmail dot com
-Status:           Feedback
+Status:           Bogus
 Bug Type:         LDAP related
 Operating System: unix
 PHP Version:      5.3SVN-2009-10-08 (snap)
 Assigned To:      srinatar
 New Comment:

Jani
 you are right. using autoconf 2.13 made this problem go away. i should
haved listened to the warning when i ran buildconf.


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

[2009-10-08 10:09:57] j...@php.net

I have objection because that patch won't work. And it's exactly same
thing as what is already done in ext/ldap/config.m4.

What was the configure line you used? Are you using proper autoconf
version (only supported and working one is 2.13!)

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

[2009-10-08 07:58:06] srina...@php.net

here is a patch that addresses this issue
[sn123...@mbelshe]'php-5.2.11'>diff -u ext/ldap/config.m4.ORIG
ext/ldap/config.m4
--- ext/ldap/config.m4.ORIG     Wed Oct  7 23:58:39 2009
+++ ext/ldap/config.m4  Thu Oct  8 00:37:21 2009
@@ -181,7 +181,10 @@
 
   dnl Solaris 2.8 claims to be 2004 API, but doesn't have
   dnl ldap_parse_reference() nor ldap_start_tls_s()
-  AC_CHECK_FUNCS([ldap_parse_result ldap_parse_reference
ldap_start_tls_s])
+  dnl Verify whether the LDAP_DIR includes these ldap functions.
+  AC_CHECK_LIB([ldap_parse_result], [ldap], [HAVE_LDAP_PARSE_RESULT])
+  AC_CHECK_LIB([ldap_parse_reference], [ldap],
[HAVE_LDAP_PARSE_REFERENCE])
+  AC_CHECK_LIB([ldap_start_tls_s], [ldap], [HAVE_LDAP_START_TLS_S])
   
   dnl
   dnl SASL check

if no one has objections to this suggested patch, i can commit it

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

[2009-10-08 07:56:21] sriram dot natarajan at gmail dot com

Description:
------------
php ldap extension's config.m4 when enabled with ldap extension checks
to see if certain ldap functions are to be enabled. currently, these
ldap functions are checked to be available on the system rather than
with the ldap library that the customer is trying to compile with

for example, if i try to php with openldap on solaris or aix, current
ldap's config.m4 extension uses AC_CHECK_FUNCS to determine if these
functions are available on the system . this is ok, if ldap library is
available on the system (like is the case with linux)

better solution would be to check if these functions are avaialable
with the ldap library provided at this option : --with-ldap=<directory>


Reproduce code:
---------------
compiling with --with-ldap=<path-to-openldap> on aix or solaris machine
causes these ldap functions to be not available even though opendlap
library has them

Expected result:
----------------
ldap_parse_result, ldap_start_tls_s functions should be available on
any unix systems if compiled against openldap library

Actual result:
--------------
all ldap functions are avaialable if linked against openldap


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


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

Reply via email to