GNU bug report logs -
#68803
[PATCH] Substitute all emacsclient occurrences
Previous Next
Reported by: Lin Jian <me <at> linj.tech>
Date: Mon, 29 Jan 2024 22:20:01 UTC
Severity: wishlist
Tags: patch
Fixed in version 31.1
Done: Stefan Kangas <stefankangas <at> gmail.com>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 68803 in the body.
You can then email your comments to 68803 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#68803
; Package
emacs
.
(Mon, 29 Jan 2024 22:20:01 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Lin Jian <me <at> linj.tech>
:
New bug report received and forwarded. Copy sent to
bug-gnu-emacs <at> gnu.org
.
(Mon, 29 Jan 2024 22:20:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Tags: patch
After commit "From .desktop files, reuse a frame or start a new
Emacs as required" on 2021-06-30, there are two emacsclient
occurrences for the Exec key of etc/emacsclient.desktop.
Before this change, only the first occurrence is substituted.
I propose a patch to substitute all emacsclient occurrences.
In GNU Emacs 29.1 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo
version 1.18.0, Xaw3d scroll bars)
Windowing system distributor 'The X.Org Foundation', version 11.0.12101010
System Description: NixOS 24.05 (Uakari)
Configured using:
'configure
--prefix=/nix/store/mzdvpxzkhlryxmbxwbih7bgvkzkw1ly7-emacs-29.1
--disable-build-details --with-modules --with-x-toolkit=lucid
--with-xft --with-cairo --with-compress-install
--with-toolkit-scroll-bars --with-native-compilation
--without-imagemagick --without-small-ja-dic --with-tree-sitter
--with-xinput2 --without-xwidgets'
[0001-Substitute-all-emacsclient-occurrences.patch (text/patch, attachment)]
Severity set to 'wishlist' from 'normal'
Request was from
Stefan Kangas <stefankangas <at> gmail.com>
to
control <at> debbugs.gnu.org
.
(Tue, 30 Jan 2024 00:43:01 GMT)
Full text and
rfc822 format available.
Reply sent
to
Stefan Kangas <stefankangas <at> gmail.com>
:
You have taken responsibility.
(Wed, 12 Feb 2025 03:54:01 GMT)
Full text and
rfc822 format available.
Notification sent
to
Lin Jian <me <at> linj.tech>
:
bug acknowledged by developer.
(Wed, 12 Feb 2025 03:54:02 GMT)
Full text and
rfc822 format available.
Message #12 received at 68803-done <at> debbugs.gnu.org (full text, mbox):
Version: 31.1
Lin Jian <me <at> linj.tech> writes:
> Tags: patch
>
> After commit "From .desktop files, reuse a frame or start a new
> Emacs as required" on 2021-06-30, there are two emacsclient
> occurrences for the Exec key of etc/emacsclient.desktop.
>
> Before this change, only the first occurrence is substituted.
>
> I propose a patch to substitute all emacsclient occurrences.
Thanks for the patch, and sorry here also for the delay in reviewing.
Installed on master as commit a73a1e00523. I'm consequently closing
this bug report.
Information forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#68803
; Package
emacs
.
(Wed, 12 Feb 2025 04:28:01 GMT)
Full text and
rfc822 format available.
Message #15 received at 68803-done <at> debbugs.gnu.org (full text, mbox):
I am surprised and happy to see my two trivial patches merged one year
after they were proposed. Thank you, Stefan.
BTW, do you have any suggestions about getting patches reviewed and
merged sooner? One year for trivial patches to be merged seems a bit
long to me.
Information forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#68803
; Package
emacs
.
(Wed, 12 Feb 2025 05:08:02 GMT)
Full text and
rfc822 format available.
Message #18 received at 68803 <at> debbugs.gnu.org (full text, mbox):
Lin Jian <me <at> linj.tech> writes:
> I am surprised and happy to see my two trivial patches merged one year
> after they were proposed. Thank you, Stefan.
>
> BTW, do you have any suggestions about getting patches reviewed and
> merged sooner? One year for trivial patches to be merged seems a bit
> long to me.
Fundamentally, the reason is that we just don't have enough people on
board that are reviewing patches. We are all volunteers and Emacs is a
very large project. This means that we sometimes miss things, or let
them fall by the wayside, when we perhaps shouldn't have.
Do feel free to ping us after say a month or two, if we didn't get to
one of your patches. You can also help make our job easier by providing
additional details, such as briefly detailing any testing done, writing
a clear yet succinct rationale for why the patch is right, and so on.
Thanks again for contributing to Emacs.
Information forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#68803
; Package
emacs
.
(Wed, 12 Feb 2025 05:31:01 GMT)
Full text and
rfc822 format available.
Message #21 received at 68803 <at> debbugs.gnu.org (full text, mbox):
Stefan Kangas <stefankangas <at> gmail.com> writes:
>
> Do feel free to ping us after say a month or two, if we didn't get to
> one of your patches.
What is the right way to ping related people?
Information forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#68803
; Package
emacs
.
(Wed, 12 Feb 2025 05:51:02 GMT)
Full text and
rfc822 format available.
Message #24 received at 68803 <at> debbugs.gnu.org (full text, mbox):
Lin Jian <me <at> linj.tech> writes:
> Stefan Kangas <stefankangas <at> gmail.com> writes:
>
>>
>> Do feel free to ping us after say a month or two, if we didn't get to
>> one of your patches.
>
> What is the right way to ping related people?
Just write to the bug tracker, i.e. the NNNNN <at> debbugs.gnu.org address
that was assigned to track the patch.
If the patch is for some specific package, you can also check the
"Maintainer" at the top of the file and make sure to Cc that person.
Information forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#68803
; Package
emacs
.
(Wed, 12 Feb 2025 13:50:02 GMT)
Full text and
rfc822 format available.
Message #27 received at 68803 <at> debbugs.gnu.org (full text, mbox):
> Cc: 68803 <at> debbugs.gnu.org
> From: Stefan Kangas <stefankangas <at> gmail.com>
> Date: Tue, 11 Feb 2025 21:07:18 -0800
>
> Lin Jian <me <at> linj.tech> writes:
>
> > I am surprised and happy to see my two trivial patches merged one year
> > after they were proposed. Thank you, Stefan.
> >
> > BTW, do you have any suggestions about getting patches reviewed and
> > merged sooner? One year for trivial patches to be merged seems a bit
> > long to me.
>
> Fundamentally, the reason is that we just don't have enough people on
> board that are reviewing patches. We are all volunteers and Emacs is a
> very large project. This means that we sometimes miss things, or let
> them fall by the wayside, when we perhaps shouldn't have.
That is true, but I think in this particular case the main problem is
that we have zero people on board who know enough about these desktop
files and routinely review patches in that area.
IOW, the time it takes to review a patch depends on the relevant
expertise we have handy much more than on the number of people who
routinely review patches. Patches in many other areas are reviewed
and acted upon within days.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Thu, 13 Mar 2025 11:24:06 GMT)
Full text and
rfc822 format available.
This bug report was last modified today.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.