[Geany-Devel] Spawn module API

[Lex Trotman]

On 24 June 2015 at 10:06, Colomban Wendling <lists.ban at herbesfolles.org> wrote:
> Le 24/06/2015 01:57, Lex Trotman a écrit :
>> Colomban,
>>
>> Correct me if I'm wrong, but despite my loudly voiced misgivings :)
>> doesn't the spawn_* series do command quoting and g_spawn* not?
>>
>> If that the case they should not be mixed on the same platform
>> otherwise the user has to know which commands are run with which
>> function to know if they need to do the quoting themselves.
>
> Well, those support an additional "command" parameter that indeed is
> read in a platform-dependent manner.  (this was a discussion point and
> ended like this).
>
> They however also support argvs just fine, so you can use those too and
> they would work the same.  With this, you can also imagine using
> g_shell_parse_argv() on all platforms if you like, just like you would
> do with g_spawn().

Then Geany should be changed to do that too, for all commands (IIRC
build commands already do it).

Note by "user" I mean the end user, not the plugin programmer.  Having
the end user need to know if they should quote commands or not is bad
(tm).  If that is already the case then it should be fixed.

Cheers
Lex


>
> So well, yes, the user has to know which syntax is needed, but that's
> basically true already.
>
> Cheers,
> Colomban
> _______________________________________________
> Devel mailing list
> Devel at lists.geany.org
> https://lists.geany.org/cgi-bin/mailman/listinfo/devel