ID: 33251 Updated by: [EMAIL PROTECTED] -Summary: build system Reported By: ohmer at epita dot info -Status: Open +Status: Closed -Bug Type: Unknown/Other Function +Bug Type: *Compile Issues Operating System: windows PHP Version: 5.0.4 New Comment:
Merged your changes into HEAD (will be PHP 5.1). Thanks for letting me know. Note that we use a different, more encompassing, win32 build collection on the snapshots machine; that includes working pgsql headers and libraries, so the search path there isn't redundant at all. http://viewcvs.php.net/viewcvs.cgi/php-src/win32/build/confutils.js.diff?r1=1.55&r2=1.56 this patch should correct your fd-setsize issue, which appears to be a side effect of using --disable-all. Previous Comments: ------------------------------------------------------------------------ [2005-06-05 20:31:50] ohmer at epita dot info Description: ------------ hello, i achieved to build php 5.0.4 on windows. i am submitting a bug report to the build system because i have found some errors in it. my configure line was : C:\php-5.0.4>cscript /nologo configure.js --disable-all --with-extra-includes=C:/Apache2/include --with-extra-libs="C:/Apache2/lib" --enable-apache2handler --with-bz2 --enable-ftp --enable-spl --enable-zlib --enable-session --with-pgsql="C:\Program Files\PostgreSQL\8.0" --with-mysql="C:\Program Files\MySQL\MySQL Server 4.1" --with-mysqli="C:\Program Files\MySQL\MySQL Server 4.1" --with-mime-magic --with-sqlite --with-openssl --enable-fd-setsize=256 1) not using --enable-set-fd=256 causes invalid CFLAGS generation seemingly because of the call to parseInt fails (see win32/build/config.w32 line 228) returning NaN. 2) i had to adjust ext/{mysql,mysqli,pgsql}/config.w32 to make it correctly find my headers and libraries : mysql: CHECK_HEADER_ADD_INCLUDE("mysql.h", "CFLAGS_MYSQL", PHP_MYSQL + "\\include;" + PHP_PHP_BUILD + "\\include")) mysqli: CHECK_HEADER_ADD_INCLUDE("mysql.h", "CFLAGS_MYSQLI", PHP_MYSQLI + "\\include;" + PHP_PHP_BUILD + "\\include")) pgsql: CHECK_HEADER_ADD_INCLUDE("libpq-fe.h", "CFLAGS_PGSQL", PHP_PGSQL + "\\include;" + PHP_PHP_BUILD + \\include\\pgsql")) as you will see while diffing with tarball's version, these modifications are only inverting search order and correcting them to follow concerned windows distribution layout. notice that the second search path for postgresql is useless as the win32build.zip provided does not contain pgsql headers nor lib. thanks for this new configure-like build system, this is really helpful and friendly to unix users like me. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=33251&edit=1
