GNU bug report logs - #55149
29.0.50; Commit f30625943e broke magit/with-editor

Previous Next

Package: emacs;

Reported by: Tassilo Horn <thorn <at> fastmail.fm>

Date: Wed, 27 Apr 2022 12:59:01 UTC

Severity: normal

Tags: patch

Merged with 55160, 55161

Found in version 29.0.50

Done: Stefan Monnier <monnier <at> iro.umontreal.ca>

Bug is archived. No further changes may be made.

Full log


Message #23 received at 55149 <at> debbugs.gnu.org (full text, mbox):

From: Tassilo Horn <thorn <at> fastmail.fm>
To: "M. Ian Graham" <hello <at> miangraham.com>
Cc: "55149 <at> debbugs.gnu.org" <55149 <at> debbugs.gnu.org>,
 "monnier <at> iro.umontreal.ca" <monnier <at> iro.umontreal.ca>
Subject: Re: bug#55149: 29.0.50; Commit f30625943e broke magit/with-editor
Date: Thu, 28 Apr 2022 08:28:37 +0200
"M. Ian Graham" <hello <at> miangraham.com> writes:

>> Oh, you are right. But I was able to extract (at least one of) the
>> problematic with-editor advice(s) which cause the breakage.
>
>> Save the below as bug-recipe.el:
>
> Nice find. Confirmed on my end that this breaks emacsclient commands
> after f30625943e but not before. Had to add a require 'cl-lib to the
> top, otherwise works perfectly with bare emacs -Q.

Oh, well, the recipe can even be simplyfied to defining just a
do-nothing advice!

--8<---------------cut here---------------start------------->8---
(advice-add 'server-visit-files :after
            #'server-visit-files--with-editor-file-name-history-exclude)

(defun server-visit-files--with-editor-file-name-history-exclude
    (files _proc &optional _nowait)
  nil)

(server-start)
--8<---------------cut here---------------end--------------->8---

Bye,
Tassilo




This bug report was last modified 3 years and 24 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.