unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Robert Pluim <rpluim@gmail.com>
To: 74775@debbugs.gnu.org
Cc: bjorn.bidar@thaodan.de, eliz@gnu.org
Subject: bug#74775: [PATCH] Update browse-url-kde variables
Date: Wed, 11 Dec 2024 15:28:21 +0100	[thread overview]
Message-ID: <87a5d25m62.fsf@gmail.com> (raw)
In-Reply-To: <87cyhyuxjh.fsf@thaodan.de> ("Björn Bidar via \"Bug reports for GNU Emacs, the Swiss army knife of text editors\""'s message of "Wed, 11 Dec 2024 16:03:30 +0200")

>>>>> On Wed, 11 Dec 2024 16:03:30 +0200, Björn Bidar via "Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org> said:

    Björn> Eli Zaretskii <eliz@gnu.org> writes:
    >>> Date: Wed, 11 Dec 2024 01:04:56 +0200
    >>> From:  Björn Bidar via "Bug reports for GNU Emacs,
    >>> the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
    >>> 
    >>> Update browse-url-kde's variables.
    >> 
    >> Thanks, but when we change the default value of a user option, we need
    >> to update its :version tag (so that commands like customize-changed
    >> could notice that).

    Björn> Will do good call.

I already did that this morning 😀

    >>> :type 'string
    >> 
    >> Should this be 'file instead?

    Björn> Does 'file for programs in path?

Yes (although Iʼm not sure customize actually checks that)

    Björn> I think it would make sense to apply this change to Emacs 30 too as it
    Björn> isn't a functionality change but a fix to not use kfmclient which is not
    Björn> installed anymore these days as the default file manager for KDE is
    Björn> Dolphin and not Konqueror which includes kfmclient.

If I squint I could call it a regression, but Iʼm not an Emacs
maintainer 😺

Robert
-- 





  reply	other threads:[~2024-12-11 14:28 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <8734ivrvfr.fsf@>
2024-12-11 12:38 ` bug#74775: [PATCH] Update browse-url-kde variables Eli Zaretskii
2024-12-11 14:03   ` Björn Bidar via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-12-11 14:28     ` Robert Pluim [this message]
2024-12-11 16:14     ` Eli Zaretskii
2024-12-11 17:41       ` Björn Bidar via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-12-11 19:07         ` Eli Zaretskii
2024-12-12 16:44           ` Björn Bidar via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-12-12 18:53             ` Eli Zaretskii
     [not found] <6758c999.050a0220.38af58.bd51SMTPIN_ADDED_BROKEN@mx.google.com>
2024-12-11  1:22 ` Stefan Kangas
2024-12-10 23:04 Björn Bidar via Bug reports for GNU Emacs, the Swiss army knife of text editors

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87a5d25m62.fsf@gmail.com \
    --to=rpluim@gmail.com \
    --cc=74775@debbugs.gnu.org \
    --cc=bjorn.bidar@thaodan.de \
    --cc=eliz@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).