[PATCH 3/6] use CreateProcess instead of system to run scripts.

[David Woodhouse]

On Thu, 2014-08-21 at 17:20 +0200, Nikos Mavrogiannopoulos wrote:
> That prevents the pop up of terminal windows.


> 
> Replacing "cmd.exe /c" with "cscript.exe" seem to work. Though we'd
> lose the ability to run anything else than script. Don't know if that
> matters though. The speed difference seems insignificant.

I'm not really worried about the speed; more about reliability and
escaping strings. It's possible to spawn whatever you like from a
script, isn't it? So I suspect it should be fine to do that.

If a caller really wants to do the setup in C code (and thus would be
inclined to put it in an executable like the NetworkManager helper), the
library allows them to do that *in* their application instead of having
to be a child process anyway.

-- 
dwmw2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 5745 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/openconnect-devel/attachments/20140826/bda912ab/attachment.bin>