[Buildbot-devel] win32 create process info

[Cory Dodt]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

The correct command line is "cmd.exe /c ..." (equivalent to sh -e) which means
/c needs to be added also.  I don't know of any downside to this; cmd loads
its environment from the registry, the same as all other windows programs.

However, this is not what I actually do on our internal systems.  I use
exemaker @ <http://effbot.org/zone/exemaker.htm>.  I will probably end up
changing Twisted's win32 installer to distribute exemaker'd versions of the
scripts so the gymnastics performed by twistenv.bat are no longer necessary.

Still, fixing Twisted's installer doesn't solve buildbot's problem when
running other python scripts in general, so cmd /c is a better solution.

C


Brian Warner wrote:
|>So, to summarize, it seems that if buildbot is going to use the win32
|>reactor then either full paths will be needed or the command call will
|>have to be changed to have cmd.exe be the application name and trial be
|>part of the command line parameters.  cmd.exe searches the path and
|>honors the PATHEXT settings.
|
|
| Is this sort of a windows equivalent to the unix "/bin/sh -e command argv.."
| pattern? If so, do you think it would work to have the windows buildslave
| just do 'command.insert(0, "cmd.exe")' on the array of strings being passed
| over from the buildmaster?
|
| If that lets command[0] be searched for on the PATH, and doesn't require
| shell-escaping spaces and stuff, maybe it would be the best answer. You
| wouldn't normally do that on the unix side because you'd have to specify the
| arguments as one flat string instead of an array of argv values.. is there
| any down-side to always using cmd.exe under windows?
|
| thanks,
|  -Brian
|
|
| -------------------------------------------------------
| SF email is sponsored by - The IT Product Guide
| Read honest & candid reviews on hundreds of IT Products from real users.
| Discover which products truly live up to the hype. Start reading now.
| http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
| _______________________________________________
| Buildbot-devel mailing list
| Buildbot-devel at lists.sourceforge.net
| https://lists.sourceforge.net/lists/listinfo/buildbot-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (MingW32)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFCbmaE3A5SrXAiHQcRAlrvAJ9tRoxGiOTRRz7HrGp1yEsraW96BwCgl7Kt
uf6fDTFea0GHFP0igq5nNqw=
=FuaP
-----END PGP SIGNATURE-----