#43281 [Opn->Csd]: Allow buildconf to pass parameters to cscript

From: Date: Wed, 14 Nov 2007 01:43:22 +0000
Subject: #43281 [Opn->Csd]: Allow buildconf to pass parameters to cscript
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-119621@lists.php.net to get a copy of this message
 ID:               43281
 Updated by:       johannes@php.net
 Reported By:      ezyang@php.net
-Status:           Open
+Status:           Closed
 Bug Type:         *Compile Issues
 Operating System: Windows
 PHP Version:      5.2CVS-2007-11-13 (CVS)
 New Comment:

This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.




Previous Comments:
------------------------------------------------------------------------

[2007-11-13 21:51:11] ezyang@php.net

Description:
------------
The buildconf.js script allows for the --add-modules-dir argument to
add scan directories for config.w32 files. However, the convenience
buildconf.bat script does not pass on configuration directives. This can
be fixed by adding %* to the end of the call:

Index: buildconf.bat
===================================================================
RCS file: /repository/php-src/buildconf.bat,v
retrieving revision 1.1
diff -u -r1.1 buildconf.bat
--- buildconf.bat	2 Dec 2003 23:16:47 -0000	1.1
+++ buildconf.bat	13 Nov 2007 21:41:01 -0000
@@ -1,2 +1,2 @@
 @echo off
-cscript /nologo win32\build\buildconf.js
+cscript /nologo win32\build\buildconf.js %*


Reproduce code:
---------------
> cd
C:\php\src\php5
> buildconf --add-modules-dir=C:\php\src\pecl


Expected result:
----------------
Adding C:\php\src\pecl to module search path

Actual result:
--------------
Module not added


------------------------------------------------------------------------


-- 
Edit this bug report at http://bugs.php.net/?id=43281&edit=1


Thread (2 messages)

« previous php.bugs (#119621) next »