Apologies if this got lost amongst the effort to prepare the recent set of releases, but I'm curious whether or not people find my second patch acceptable, based on the results of the prior discussion. If not, I'll know to abandon the issue.
Here is a patch that we plan on using at work (Y!). This patch has two key additions to Jon's patch. The first I think should be added to Jon's patch so that shared extensions (phpize) build with his environment variable change and the second is a change so we don't have to set environment variables and it will pick up the right files. The second change may or may not make it in, but I think the first one should.
The first change is in scripts/phpize.in where it tests and set PHP_AUTOCONF/PHP_AUTOHEADER variables, then uses them below. This patch should be added without the:
+ if ! which $PHP_AUTOCONF; then
+ PHP_AUTOCONF=`ls -1 /usr/local/bin/autoconf* | sort -r | head -1`;
+ PHP_AUTOHEADER=`ls -1 /usr/local/bin/autoheader* | sort -r | head -1`;
+ fi
+
block in it (which would be part of the second change).
The second changes are the ones that do a which command to see if PHP_AUTOCONF/PHP_AUTOHEADER is really valid and if not try to find the valid one. I don't know how compatible this is across os's and is the reason I can see it not making it (but it works for us). This package also allows us to use any version of the ports/autoconf package without having to set environment variable.
Let me know what you think.
Thanks,
Brian
-- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
