Subject: Re: bug#74775: [PATCH] Update browse-url-kde variables
From: Stefan Kangas <[email-masked]>
To: Björn Bidar <[email-masked]>, [email-masked]
Date: Tue, 10 Dec 2024 20:22:00 -0500
List-Id: <debbugs-submit.debbugs.gnu.org>

Björn Bidar via "Bug reports for GNU Emacs, the Swiss army knife of text
editors" <[email-masked]> writes:

> From 412418551717c788320a3d1f04a68f47f263c968 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Bj=C3=B6rn=20Bidar?= <[email-masked]>
> Date: Wed, 11 Dec 2024 00:43:29 +0200
> Subject: [PATCH] Update browse-url-kde variables
>
> * lisp/browse-url.el (browse-url-kde-program)
> (browse-url-kde-args):
> Update defaults to more modern alternative to kfmclient.
> Kfmclient is only installed when Konqueror is installed which isn't the
> default file-manager since KDE4.
> In contrast it is more likely that kde-open is installed as it is
> required by Plasma, it exists since KDE4.
> `browse-url-kde-args` isn't required anymore but kept to not break
> compatibility.
> ---
>  lisp/net/browse-url.el | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lisp/net/browse-url.el b/lisp/net/browse-url.el
> index c10bc671a88..5a54d653720 100644
> --- a/lisp/net/browse-url.el
> +++ b/lisp/net/browse-url.el
> @@ -517,12 +517,12 @@ browse-url-text-input-delay
>    :type 'number
>    :version "23.1")
>
> -(defcustom browse-url-kde-program "kfmclient"
> +(defcustom browse-url-kde-program "kde-open"
>    "The name by which to invoke the KDE web browser."
>    :type 'string
>    :version "21.1")
>
> -(defcustom browse-url-kde-args '("openURL")
> +(defcustom browse-url-kde-args nil
>    "A list of strings defining options for `browse-url-kde-program'."
>    :type '(repeat (string :tag "Argument")))
>
> --
> 2.45.2

Thanks, installed on master as commit 63b9ddda8f8.

I'm therefore closing this bug report.