[prev in list] [next in list] [prev in thread] [next in thread]
List: openconnect-devel
Subject: [PATCH 3/6] use CreateProcess instead of system to run scripts.
From: dwmw2 () infradead ! org (David Woodhouse)
Date: 2014-08-26 10:42:14
Message-ID: 1409049734.29915.27.camel () i7 ! infradead ! org
[Download RAW message or body]
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>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic