Use COMSPEC command interpreter to run .bat files.

[Sylvain Petreolle]

Is your patch allowing the run of .bat files by wine itself, or
only a W-process ? When I try it on command line I have this :

[syl@snoop cygwin]$ wine cygwin.bat --debugmsg +process
trace:process:PROCESS_InitWine starting process name="" file=(nil)
argv[0]="cygwin.bat"
trace:process:find_exe_file looking for "cygwin.bat"
trace:process:find_exe_file Trying built-in exe
"C:\cygwin\cygwin.bat"
trace:process:find_exe_file Trying native exe "C:\cygwin\cygwin.bat"
wine: cannot determine executable type for 'C:\cygwin\cygwin.bat'
[syl@snoop cygwin]$ cat cygwin.bat
@echo off

C:
chdir \cygwin\bin

bash --login -i

=====
Sylvain Petreolle
[e-mail] 
Fight against Spam ! http://www.euro.cauce.org/en/index.html
ICQ #170597259

"Don't think you are. Know you are." Morpheus in Matrix, chapter 15.

___________________________________________________________
Soyez solidaire soutenez l’action du Téléthon avec Yahoo! France.
http://www1.telethon.fr/030-Espace-Relais-Dons/webtirelire1.asp?hebergeur_id...

[Uwe Bonnes]

> > > > > "Sylvain" == Sylvain Petreolle  writes:

Sylvain> Is your patch allowing the run of .bat files by wine itself, or
    Sylvain> only a W-process ? When I try it on command line I have this :

Sylvain> [syl@snoop cygwin]$ wine cygwin.bat --debugmsg +process
    Sylvain> trace:process:PROCESS_InitWine starting process name=""
    Sylvain> file=(nil) argv[0]="cygwin.bat" trace:process:find_exe_file
    Sylvain> looking for "cygwin.bat" trace:process:find_exe_file Trying
    Sylvain> built-in exe "C:\cygwin\cygwin.bat"
    Sylvain> trace:process:find_exe_file Trying native exe
    Sylvain> "C:\cygwin\cygwin.bat" wine: cannot determine executable type
    Sylvain> for 'C:\cygwin\cygwin.bat' [syl@snoop cygwin]$ cat cygwin.bat
    Sylvain> @echo off

you can't run "wine cygwin.bat --debugmsg +process". Only
"CreateProcess(...,"...bat") is supported? And the fix for GOTO in batch
files needs still to be applied.

Is "wine cygwin.bat --debugmsg +process" really something valid? "wine
--debugmsg +process -- wcmd /c cygwin.bat" Should work.

Bye

[Sylvain Petreolle]

Thanks, this works.
But it seems another problem is coming.
I checked the COMSPEC value under wcmd.
It currently answers COMSPEC=C:\WINDOWS\SYSTEM\wcmd.exe.

This cannot be found - wineinstall sets links into system32.
  link_app wcmd         "$CROOT/windows/system32/cmd.exe"
Is wcmd setting comspec ? If so, this setting must be changed.
> you can't run "wine cygwin.bat --debugmsg +process". Only
> "CreateProcess(...,"...bat") is supported? And the fix for GOTO in
> batch
> files needs still to be applied.
>
> Is "wine cygwin.bat --debugmsg +process" really something valid?
> "wine
> --debugmsg +process -- wcmd /c cygwin.bat" Should work.

=====
Sylvain Petreolle
[e-mail] 
Fight against Spam ! http://www.euro.cauce.org/en/index.html
ICQ #170597259

"Don't think you are. Know you are." Morpheus in Matrix, chapter 15.

___________________________________________________________
Soyez solidaire soutenez l’action du Téléthon avec Yahoo! France.
http://www1.telethon.fr/030-Espace-Relais-Dons/webtirelire1.asp?hebergeur_id...

[Uwe Bonnes]

> > > > > "Sylvain" == Sylvain Petreolle  writes:

Sylvain> Thanks, this works.  But it seems another problem is coming.  I
    Sylvain> checked the COMSPEC value under wcmd.  It currently answers
    Sylvain> COMSPEC=C:\WINDOWS\SYSTEM\wcmd.exe.

Sylvain> This cannot be found - wineinstall sets links into system32.
    Sylvain> link_app wcmd "$CROOT/windows/system32/cmd.exe" Is wcmd setting
    Sylvain> comspec ? If so, this setting must be changed.

>> you can't run "wine cygwin.bat --debugmsg +process". Only
    >> "CreateProcess(...,"...bat") is supported? And the fix for GOTO in
    >> batch files needs still to be applied.
    >> 
    >> Is "wine cygwin.bat --debugmsg +process" really something valid?
    >> "wine --debugmsg +process -- wcmd /c cygwin.bat" Should work.

If you run "wine --debugmsg +process -- wcmd /c cygwin.bat", my patch
shouldn't be involved. Is in your case a 'CreateProcess (...,"....bat
...")' somehow involved? Only then my patch should change anything. 
Wcmd should be found somehow magically, either in ../wine/programs or by the
COMSPEC variable.

Where does the wine install process put WCMD? As I always run wine in my
build tree, I can't check...

Bye

[Sylvain Petreolle]

Here is what I get in wcmd :
C>dir cygwin.bat                                  
Volume in drive C is                              
Volume Serial Number is 1234-5678

Directory of C:\cygwin\

01/03/2002  05:05:16            55  cygwin.bat    
       1 file                        55 bytes     
       0 directories      2,218,336,256 bytes free

C>wcmd /c cygwin.bat                              
File not found

C>
> If you run "wine --debugmsg +process -- wcmd /c cygwin.bat", my patch
> shouldn't be involved. Is in your case a 'CreateProcess (...,"....bat
> ...")' somehow involved? Only then my patch should change anything. 
> Wcmd should be found somehow magically, either in ../wine/programs or
> by the
> COMSPEC variable.

link_app wcmd "$CROOT/windows/system32/cmd.exe
As shown in the above line fom wineinstall, its put in
C:\windows\system32, with some other built-in .exe.
> Where does the wine install process put WCMD? As I always run wine in
> my
> build tree, I can't check...

=====
Sylvain Petreolle
[e-mail] 
Fight against Spam ! http://www.euro.cauce.org/en/index.html
ICQ #170597259

"Don't think you are. Know you are." Morpheus in Matrix, chapter 15.

___________________________________________________________
Soyez solidaire soutenez l’action du Téléthon avec Yahoo! France.
http://www1.telethon.fr/030-Espace-Relais-Dons/webtirelire1.asp?hebergeur_id...

[Uwe Bonnes]

> > > > > "Sylvain" == Sylvain Petreolle  writes:

Sylvain> Here is what I get in wcmd :
    C> dir cygwin.bat
    Sylvain> Volume in drive C is Volume Serial Number is 1234-5678

Sylvain> Directory of C:\cygwin\

Sylvain> 01/03/2002 05:05:16 55 cygwin.bat 1 file 55 bytes 0 directories
    Sylvain> 2,218,336,256 bytes free

C> wcmd /c cygwin.bat
    Sylvain> File not found
> From inside wcmd you can run the batch directly. In this case, th epatch

isn't involed neither.

Try
C> cygwin.bat

Bye

[Sylvain Petreolle]

I known it works directly, but it should work with wcmd...
> From inside wcmd you can run the batch directly. In this case, th
> epatch
> isn't involed neither.
>
> Try
> C> cygwin.bat

=====
Sylvain Petreolle
[e-mail] 
Fight against Spam ! http://www.euro.cauce.org/en/index.html
ICQ #170597259

"Don't think you are. Know you are." Morpheus in Matrix, chapter 15.

___________________________________________________________
Soyez solidaire soutenez l’action du Téléthon avec Yahoo! France.
http://www1.telethon.fr/030-Espace-Relais-Dons/webtirelire1.asp?hebergeur_id...

[Uwe Bonnes]

> > > > > "Sylvain" == Sylvain Petreolle  writes:

Sylvain> I known it works directly, but it should work with wcmd...
    >> From inside wcmd you can run the batch directly. In this case, th
    >> epatch isn't involed neither.
    >> 
    >> Try
    C> cygwin.bat

To parse inside wcmd

c> wcmd /c cygwin.bat

we need to have a look at wcmd itself. Probably it doesn't know itself :-)

Bye

[Sylvain Petreolle]

I found why we have a 'not found', the problem is that wcmd isn't
installed by default in c:\windows\system32, it installed as "cmd".
should we add it to wineinstall or what ?

for the false COMSPEC problem, I found it too.
it is in files/directory.c:107/180.
it assigns wcmd as interpreter.

to give a summary:
wineinstall installs cmd.exe in windows/system32.
COMSPEC is set to search wcmd.exe => we need to add it in wineinstall.
COMSPEC is set find it in windows/system => we should change it in
files/directory.c to point to windows/system32 instead.

Comments ?
> To parse inside wcmd
>
> c> wcmd /c cygwin.bat
>
> we need to have a look at wcmd itself. Probably it doesn't know
> itself :-)

=====
Sylvain Petreolle
[e-mail] 
Fight against Spam ! http://www.euro.cauce.org/en/index.html
ICQ #170597259

"Don't think you are. Know you are." Morpheus in Matrix, chapter 15.

___________________________________________________________
Soyez solidaire soutenez l’action du Téléthon avec Yahoo! France.
http://www1.telethon.fr/030-Espace-Relais-Dons/webtirelire1.asp?hebergeur_id...

[Uwe Bonnes]

> > > > > "Sylvain" == Sylvain Petreolle  writes:

Sylvain> I found why we have a 'not found', the problem is that wcmd
    Sylvain> isn't installed by default in c:\windows\system32, it installed
    Sylvain> as "cmd".  should we add it to wineinstall or what ?

Sylvain> for the false COMSPEC problem, I found it too.  it is in
    Sylvain> files/directory.c:107/180.  it assigns wcmd as interpreter.

Sylvain> to give a summary: wineinstall installs cmd.exe in
    Sylvain> windows/system32.  COMSPEC is set to search wcmd.exe => we need
    Sylvain> to add it in wineinstall.  COMSPEC is set find it in
    Sylvain> windows/system => we should change it in files/directory.c to
    Sylvain> point to windows/system32 instead.

It's tools/wineinstall that does rename wcmd.exe to cmd.exe when installing. 
I don't think this is right. Other opinions?

Bye

[Alexandre Julliard]

Uwe Bonnes  writes:
> It's tools/wineinstall that does rename wcmd.exe to cmd.exe when installing. 
> I don't think this is right. Other opinions?

Actually it could be argued that this is the right thing to do, since
wcmd is supposed to be a replacement for cmd.exe. But then of course
we should rename it consistently everywhere, not just in wineinstall.

[Alexandre Julliard]

Sylvain Petreolle  writes:
> COMSPEC is set find it in windows/system => we should change it in
> files/directory.c to point to windows/system32 instead.

COMSPEC is set in directory.c to point to the windows system
directory, this is correct and should not be changed. The bug is that
wineinstall puts stuff in windows/system32 but specifies
windows/system as system dir in the config file.

[Sylvain Petreolle]

Ok, in this case we must change all stuff going into system32,
as the sample config file specifies c:\windows\system as path too.
> COMSPEC is set in directory.c to point to the windows system
> directory, this is correct and should not be changed. The bug is that
> wineinstall puts stuff in windows/system32 but specifies
> windows/system as system dir in the config file.

Index: wineinstall
===================================================================
RCS file: /home/wine/wine/tools/wineinstall,v
retrieving revision 1.49
diff -u -r1.49 wineinstall
--- wineinstall 22 Nov 2002 20:44:13 -0000      1.49
+++ wineinstall 5 Dec 2002 02:07:10 -0000
@@ -132,14 +132,14 @@
   link_app notepad      "$CROOT/windows/notepad.exe"
   link_app regedit      "$CROOT/windows/regedit.exe"
   link_app rundll32     "$CROOT/windows/rundll32.exe"
-  link_app wcmd         "$CROOT/windows/system32/cmd.exe"
-  link_app control      "$CROOT/windows/system32/control.exe"
-  link_app winhelp      "$CROOT/windows/system32/help.exe"
-  link_app notepad      "$CROOT/windows/system32/notepad.exe"
-  link_app progman      "$CROOT/windows/system32/progman.exe"
-  link_app regsvr32     "$CROOT/windows/system32/regsvr32.exe"
-  link_app winemine     "$CROOT/windows/system32/winmine.exe"
-  link_app winver       "$CROOT/windows/system32/winver.exe"
+  link_app wcmd         "$CROOT/windows/system/wcmd.exe"
+  link_app control      "$CROOT/windows/system/control.exe"
+  link_app winhelp      "$CROOT/windows/system/help.exe"
+  link_app notepad      "$CROOT/windows/system/notepad.exe"
+  link_app progman      "$CROOT/windows/system/progman.exe"
+  link_app regsvr32     "$CROOT/windows/system/regsvr32.exe"
+  link_app winemine     "$CROOT/windows/system/winmine.exe"
+  link_app winver       "$CROOT/windows/system/winver.exe"
   link_app uninstaller  "$CROOT/windows/uninstall.exe"
   link_app winhelp      "$CROOT/windows/winhelp.exe"
   link_app winhelp      "$CROOT/windows/winhlp32.exe"

=====
Sylvain Petreolle
[e-mail] 
Fight against Spam ! http://www.euro.cauce.org/en/index.html
ICQ #170597259

"Don't think you are. Know you are." Morpheus in Matrix, chapter 15.

___________________________________________________________
Soyez solidaire soutenez l’action du Téléthon avec Yahoo! France.
http://www1.telethon.fr/030-Espace-Relais-Dons/webtirelire1.asp?hebergeur_id...

[davep]

On Wednesday 04 Dec 2002 09:48, Sylvain Petreolle wrote:
> Here is what I get in wcmd :
> C>dir cygwin.bat
> Volume in drive C is
> Volume Serial Number is 1234-5678
>
> Directory of C:\cygwin\
>
> 01/03/2002  05:05:16            55  cygwin.bat
>        1 file                        55 bytes
>        0 directories      2,218,336,256 bytes free
>
> C>wcmd /c cygwin.bat
> File not found
>
> C>

For that to work wcmd needs to be able to find itself (ie it must be in one 
of the places Win32 searches for executables). wcmd has no special knowledge 
of its own name or how to invoke itself.

However even if wcmd is able to start another version of itself there is 
another problem. wcmd started in this way does not inherit the current 
directory from its parent, so unless you included the full path to 
cygwin.bat the child wcmd would not find it.

Dave
.

[Dimitrie O. Paun]

On December 4, 2002 12:26 pm, davep wrote:
> wcmd started in this way does not inherit the current
> directory from its parent

And isn't this a problem?