PHP :: Bug #38290 :: configure script ignores --without-cdb,inifile,flatfile

Bug #38290	configure script ignores --without-cdb,inifile,flatfile
Submitted:	2006-08-02 06:26 UTC	Modified:	2006-08-07 07:58 UTC
From:	tietew at tietew dot net	Assigned:	helly (profile)
Status:	Closed	Package:	*Configuration Issues
PHP Version:	5.1.*	OS:	Linux
Private report:	No	CVE-ID:	None

[2006-08-02 06:26 UTC] tietew at tietew dot net

Description:
------------
I want to compile PHP with dba but I don't want to include cdb support in order to resolve libcdb confliction problem temporarily.

I passed "--enable-dba --without-cdb" to configure script but the script have ignored --without-cdb.


Reproduce code:
---------------
./configure --enable-dba --with-gdbm --with-db4 --without-cdb

Expected result:
----------------
PHP is compiled with dba but without cdb support.

Actual result:
--------------
PHP is compiled with cdb support.

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports

[2006-08-02 09:22 UTC] tony2001@php.net

Marcus. from what I can see, you enable most of the dba options even if they were explicitly disabled:

Index: ext/dba/config.m4
===================================================================
RCS file: /repository/php-src/ext/dba/config.m4,v
retrieving revision 1.70.2.2
diff -u -p -d -r1.70.2.2 config.m4
--- ext/dba/config.m4   29 Nov 2005 18:25:58 -0000      1.70.2.2
+++ ext/dba/config.m4   2 Aug 2006 09:21:07 -0000
@@ -463,7 +463,7 @@ AC_DEFUN([PHP_DBA_BUILTIN_CDB],[

 AC_ARG_WITH(cdb,
 [  --with-cdb[=DIR]          DBA: Include CDB support],[
-  if test "$withval" = "yes" || test "$HAVE_DBA" = "1"; then
+  if test "$withval" = "yes"; then
     PHP_DBA_BUILTIN_CDB
   elif test "$withval" != "no"; then
     PHP_DBA_STD_BEGIN
@@ -493,7 +493,7 @@ AC_ARG_WITH(cdb,
     PHP_DBA_STD_ATTACH
   fi
 ],[
-  if test "$PHP_DBA" != "no" || test "$HAVE_DBA" = "1"; then
+  if test "$PHP_DBA" != "no"; then
     PHP_DBA_BUILTIN_CDB
   fi
 ])
@@ -511,7 +511,7 @@ AC_ARG_WITH(inifile,
     PHP_DBA_BUILTIN_INI
   fi
 ],[
-  if test "$PHP_DBA" != "no" || test "$HAVE_DBA" = "1"; then
+  if test "$PHP_DBA" != "no"; then
     PHP_DBA_BUILTIN_INI
   fi
 ])
@@ -532,7 +532,7 @@ AC_ARG_WITH(flatfile,
     PHP_DBA_BUILTIN_FLATFILE
   fi
 ],[
-  if test "$PHP_DBA" != "no" || test "$HAVE_DBA" = "1"; then
+  if test "$PHP_DBA" != "no"; then
     PHP_DBA_BUILTIN_FLATFILE
   fi
 ])

[2006-08-07 07:57 UTC] helly@php.net

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-08-07 07:58 UTC] helly@php.net

Btw, you only get the built-in cdb which should be ok.