Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-11 Thread Steph Fox
I am right :-) Plus, Richard improved my suggestion in private email to: $(COMSPEC) /c This would be robust against changes in future Windows versions (command.com -> cmd.exe -> whatever). Good call. I'll go through them when I get a moment. - Steph Wietse -- PHP Internals - PHP Runtim

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-11 Thread Richard Quadling
2008/5/10 Wietse Venema <[EMAIL PROTECTED]>: > Steph Fox: >> Hi Hector, >> >> > I can confirm that nmake v6 from VS C/C++ 6.00 did not exhibit the >> > problem. However nnake (v8) from VS 2005 does exhibit the problem. >> > Richard was using nmake v9 (VS 2008). >> > >> > The problem begins with ha

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-10 Thread Wietse Venema
Steph Fox: > Hi Hector, > > > I can confirm that nmake v6 from VS C/C++ 6.00 did not exhibit the > > problem. However nnake (v8) from VS 2005 does exhibit the problem. > > Richard was using nmake v9 (VS 2008). > > > > The problem begins with having /cygwin/bin folder in the PATH statement > > and

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-10 Thread Steph Fox
Hi Hector, I can confirm that nmake v6 from VS C/C++ 6.00 did not exhibit the problem. However nnake (v8) from VS 2005 does exhibit the problem. Richard was using nmake v9 (VS 2008). The problem begins with having /cygwin/bin folder in the PATH statement and RMDIR.EXE found in this folder by n

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-09 Thread Hector Santos
Richard Quadling wrote: 2008/5/9 Hector Santos <[EMAIL PROTECTED]>: nmake is the issue as it is not using the shell first. It is looking for executables via the path. System Internal's FileMonitor shows this. I can confirm that nmake v6 from VS C/C++ 6.00 did not exhibit the problem. Howev

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-09 Thread Richard Quadling
2008/5/9 Hector Santos <[EMAIL PROTECTED]>: > Richard Quadling wrote: > >> You don't actually need to install cygwin to see the fault. >> >> Create a batch file called rmdir.bat like this ... >> > > But this is not what you have in the production makefile environment. > > Here is a quick test. Crea

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-09 Thread Hector Santos
Richard Quadling wrote: You don't actually need to install cygwin to see the fault. Create a batch file called rmdir.bat like this ... But this is not what you have in the production makefile environment. Here is a quick test. Create a simple makefile in your home folder. for me, it is:

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-09 Thread Wietse Venema
Richard Quadling: > Effectively nmake is not respecting the shell to run internal commands. > > As you can see (hopefully), rmdir is a SHELL command, even if an > executable exists with the same name and that executable is in the > path. > > The issue is that nmake doesn't respect the shell comma

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-09 Thread Richard Quadling
2008/5/9 Steph Fox <[EMAIL PROTECTED]>: > Hi Richard, > >>> Wouldn't having /cygwin/bin permanently in your PATH screw up native >>> executables throughout? Can you use MS rmdir on your system at all? >>> >> No. As in normal windows, shell commands take precedence over >> programs/scripts, even wit

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-09 Thread Steph Fox
Hi Richard, Wouldn't having /cygwin/bin permanently in your PATH screw up native executables throughout? Can you use MS rmdir on your system at all? No. As in normal windows, shell commands take precedence over programs/scripts, even with the same name. If I want to use a cygwin program in pre

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-09 Thread Richard Quadling
2008/5/8 Steph Fox <[EMAIL PROTECTED]>: >> If only I could send the sound of my shoulders slumping as yet another >> post is ignored/filtered. http://news.php.net/php.internals/37517 > > I only asked why :) > >> As I said in the original post, if you have cygwin installed (as was >> needed to build

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-08 Thread Steph Fox
If only I could send the sound of my shoulders slumping as yet another post is ignored/filtered. http://news.php.net/php.internals/37517 I only asked why :) As I said in the original post, if you have cygwin installed (as was needed to build the PHP documentation in the bad old days before phd

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-08 Thread Richard Quadling
2008/5/8 Steph Fox <[EMAIL PROTECTED]>: > Hi Richard, > >> Basically search&replace rmdir with rd in the win32/build/Makefile > > Why? rd's just an alias of rmdir, and rmdir's the more human-readable of the > two. > > - Steph > If only I could send the sound of my shoulders slumping as yet another

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-08 Thread Steph Fox
Hi Richard, Basically search&replace rmdir with rd in the win32/build/Makefile Why? rd's just an alias of rmdir, and rmdir's the more human-readable of the two. - Steph -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-08 Thread Richard Quadling
2008/5/7 Hannes Magnusson <[EMAIL PROTECTED]>: > On Wed, May 7, 2008 at 6:42 PM, Richard Quadling > <[EMAIL PROTECTED]> wrote: > > Hi. > [..] > > > The attached patch forces the use of 'rd' rather than 'rmdir'. > > Only plain/text attachments get through the list.. > > -Hannes > Basically se

Re: [PHP-DEV] Win32 Makefile template patch.

2008-05-07 Thread Hannes Magnusson
On Wed, May 7, 2008 at 6:42 PM, Richard Quadling <[EMAIL PROTECTED]> wrote: > Hi. [..] > The attached patch forces the use of 'rd' rather than 'rmdir'. Only plain/text attachments get through the list.. -Hannes -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http