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
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: jd at cpanel dot net
New email:
PHP Version: OS:

 

 [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

Pull Requests

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-2025 The PHP Group
All rights reserved.
Last updated: Wed Jul 02 14:01:36 2025 UTC