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

[zhekov]

> @@ -577,7 +604,83 @@ static gboolean spawn_async_with_pipes(const gchar *working_directory, const gch
>  
>  	spawned = g_spawn_async_with_pipes(working_directory, full_argv, envp,
>  		G_SPAWN_SEARCH_PATH | (child_pid ? G_SPAWN_DO_NOT_REAP_CHILD : 0), NULL, NULL,
> -		child_pid, stdin_fd, stdout_fd, stderr_fd, error);
> +		child_pid, stdin_fd, stdout_fd, stderr_fd, &gerror);
> +
> +	if (!spawned)
> +	{
> +		gint en = 0;
> +		const gchar *message = gerror->message;
> +
> +		/* try to cut glib citing of the program name or working directory: they may be long,
> +		   and only the caller knows whether they're UTF-8. We lose the exact chdir error. */
> +		switch (gerror->code)

It's a gtk+ or glib assertion, not Geany, so we can only find some workaround.

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