[LIBREPORT PATCH] in KDE session open URLs in kde-open

Richard Marko rmarko at redhat.com
Fri Aug 9 12:43:37 UTC 2013


Works. Pushed. Thanks!

On 08/07/2013 05:48 PM, Jakub Filak wrote:
> Related to rhbz#981271
>
> Signed-off-by: Jakub Filak <jfilak at redhat.com>
> ---
>  src/gui-wizard-gtk/wizard.c | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/src/gui-wizard-gtk/wizard.c b/src/gui-wizard-gtk/wizard.c
> index 9ddf076..9a24a9e 100644
> --- a/src/gui-wizard-gtk/wizard.c
> +++ b/src/gui-wizard-gtk/wizard.c
> @@ -522,6 +522,24 @@ static void open_browse_if_link(GtkWidget *text_view, GtkTextIter *iter)
>  
>          if (url != 0)
>          {
> +            /* http://techbase.kde.org/KDE_System_Administration/Environment_Variables#KDE_FULL_SESSION */
> +            if (getenv("KDE_FULL_SESSION") != NULL)
> +            {
> +                gint exitcode;
> +                gchar *arg[3];
> +                /* kde-open is from kdebase-runtime, it should be there. */
> +                arg[0] = (char *) "kde-open";
> +                arg[1] = (char *) url;
> +                arg[2] = NULL;
> +
> +                const gboolean spawn_ret = g_spawn_sync(NULL, arg, NULL,
> +                                 G_SPAWN_SEARCH_PATH | G_SPAWN_STDOUT_TO_DEV_NULL,
> +                                 NULL, NULL, NULL, NULL, &exitcode, NULL);
> +
> +                if (spawn_ret)
> +                    break;
> +            }
> +
>              GError *error = NULL;
>              if (!gtk_show_uri(/* use default screen */ NULL, url, GDK_CURRENT_TIME, &error))
>                  error_msg("Can't open url '%s': %s", url, error->message);


-- 
Richard Marko



More information about the Crash-catcher mailing list