<p><a href="https://github.com/zhekov" class="user-mention">@zhekov</a> In my opinion, the whole prefix of the message that should be removed, i.e.:</p>

<p>search_find_in_files: spawn_with_callbacks() failed: CreateProcess() failed: </p>

<p>This is something the users shouldn't see when GError message is displayed - it's just something that might be useful for debugging (might be fine to put it to the debugging messages).</p>

<p>Regarding the grep message, I think it wasn't a good idea to replace the original error message:</p>

<pre><code>    ui_set_statusbar(TRUE, _("Cannot execute grep tool '%s';"
        " check the path setting in Preferences."), tool_prefs.grep_cmd);
</code></pre>

<p>with</p>

<pre><code>if (command_grep == NULL)
    command_line = g_strdup_printf("%s %s --", tool_prefs.grep_cmd, opts);
else
{
    command_line = g_strdup_printf("\"%s\" %s --", command_grep, opts);
</code></pre>

<p>The original message didn't use the GError at all and I think in this case it's the right thing to do - we know that the grep command failed to execute so we tell the user plus the info what needs to be changed.</p>

<p>I didn't go through all the uses of the new spawn module but I can imagine such a thing happens on other places too - from the special case where the spawn() is called we can probably create a more useful error message than what the spawn() itself can generate.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/geany/geany/issues/541#issuecomment-147085213">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ3W4i04Trj9JSrj0yB6_uc5eMH_Tks5o6Px3gaJpZM4FNtnl.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/issues/541#issuecomment-147085213"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>