php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #50337 Caching of pdo_cv_inc_path breaks compile of PDO extensions
Submitted: 2009-11-30 18:59 UTC Modified: 2009-11-30 22:13 UTC
From: jd at cpanel dot net Assigned:
Status: Closed Package: Compile Failure
PHP Version: 6 OS: linux
Private report: No CVE-ID: None
 [2009-11-30 18:59 UTC] jd at cpanel dot net
Description:
------------
Revision 291283 fixes the caching of the PDO include path, but it causes compile failures when the cached value is used.

The problem seems to be in ext/sqlite/config.m4 on line 53.

pdo_cv_inc_path="-I$pdo_cv_inc_path"

Other users of pdo_cv_inc_path expect it to be a path without the -I.  On the run of configure, when this cached value is used in ext/pdo_mysql/config.m4 it ends up with a Makefile like this:

ext/pdo_mysql/pdo_mysql.lo: /home/cpeasyapache/src/php-5.2.12RC3/ext/pdo_mysql/pdo_mysql.c
        $(LIBTOOL) --mode=compile $(CC) -I-I/home/cpeasyapache/src/php-5.2.12RC3/ext -Iext/pdo_mysql/ -I/home/cpeasyapache/src/php-5.2.12RC3/ext/pdo_mysql/ $(COMMON_FLAGS) $(CFLAGS_CLEAN) $(EXTRA_CFLAGS) -prefer-non-pic -c /home/cpeasyapache/src/php-5.2.12RC3/ext/pdo_mysql/pdo_mysql.c -o ext/pdo_mysql/pdo_mysql.lo


The -I-I<path> is the problem.


It looks like 291283 was reverted in the PHP 5.2 branch with revision 291414 but this bug still exists in trunk.


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2009-11-30 19:13 UTC] svn@php.net
Automatic comment from SVN on behalf of rasmus
Revision: http://svn.php.net/viewvc/?view=revision&revision=291517
Log: Fix bug #50337
 [2009-11-30 19:15 UTC] rasmus@php.net
Actually I hadn't noticed that ext/sqlite/config.m4 redefined 
pdo_cv_inc_path like that.  Thanks for picking that up.  That's 
probably why caching that value didn't work in the PHP_5_2 and PHP_5_3 
branches.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Tue Apr 30 04:01:30 2024 UTC