Re: cvs: php4 /pear pear.m4

From: Date: Tue, 13 Mar 2001 21:49:04 +0000
Subject: Re: cvs: php4 /pear pear.m4
References: 1 2  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-4551@lists.php.net to get a copy of this message
On Tue, Mar 13, 2001 at 10:45:45PM +0100, Stig Sæther Bakken wrote: > This is not different from the way it works in PHP. The problem is > that if you have an extension that uses "phpize" to make its own > configure script etc, and you want to define your own variables there, > configure will make a file called "php_config.h". Unless you > explicitly put "." in your include path in Makefile.in that is not the > file you get with #include "php_config.h". That include statement > will by default read $prefix/include/php/php_config.h. So to include > your module's config header file, you must do #include > "./php_config.h", which is kinda silly. > > This patch renames the _extension local_ config header to "config.h" > to clear up this confusion. And extensions themselves should be updated to use config.h after including php.h, for example, using following script: #!/bin/bash PHP4_ROOT=php4 PWD_X=pwd EXTENSIONS=find $PHP4_ROOT/ext -type d -maxdepth 1 -print for i in $EXTENSIONS ; do cd $i for j in ls *.c ; do perl -pi -e "s/#include \"php.h\"/#include \"php.h\"\n#include \"config.h\"/" $j done cd $PWD_X done -- Sincerely yours, Alexander Bokovoy The Midgard Project | www.midgard-project.org | Aurora R&D team Minsk Linux Users Group | www.minsk-lug.net | www.aurora-linux.com ALT Linux Team | www.alt-linux.org | Architecte Open Source -- Man invented language to satisfy his deep need to complain. -- Lily Tomlin

« previous php.cvs (#4551) next »