[Github-comments] [geany] Alter spawn to return the error message only in error->message (#701)

[Colomban Wendling]

>  		{
> -			ui_set_statusbar(TRUE, "Context action command failed: %s", error->message);
> +			ui_set_statusbar(TRUE, format, command, error->message);

`_("Cannot execute context action command \"%s\": %s. %s")` with a `check_msg=_("Check the path setting in Preferences.")` in each branch is probably fine for translations and works around the format issue.  @frlan what do you think, is this OK for translators?

> As of whether we really want the literal (replacement) %s - we want the original command, without the selected text, which may be very long, and is an argument anyway.

OK, if we assume it either can't be a Geany issue or that we don't care if Geany replaced `%s` in a way that broke spawning.


---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/701/files#r43426440
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20151029/fdee884f/attachment.html>