GNU bug report logs - #69809
30.0.50; flymake: error in process sentinel

Previous Next

Full log

View this message in rfc822 format

From: Eshel Yaron <me <at> eshelyaron.com>
To: Spencer Baugh <sbaugh <at> janestreet.com>, João Távora <joaotavora <at> gmail.com>
Cc: gerd.moellmann <at> gmail.com, sbaugh <at> catern.com, Eli Zaretskii <eliz <at> gnu.org>, 69809 <at> debbugs.gnu.org
Subject: bug#69809: 30.0.50; flymake: error in process sentinel
Date: Wed, 17 Jul 2024 08:12:39 +0200

Spencer Baugh <sbaugh <at> janestreet.com> writes:

> Eli Zaretskii <eliz <at> gnu.org> writes:
>
>>> From: Eshel Yaron <me <at> eshelyaron.com>
>>>
>>> [...]maybe something like the following is better:
>>> 
>>> diff --git a/lisp/progmodes/eglot.el b/lisp/progmodes/eglot.el
>>> index a893a8d749a..6cd48917d47 100644
>>> --- a/lisp/progmodes/eglot.el
>>> +++ b/lisp/progmodes/eglot.el
>>> @@ -2040,7 +2040,8 @@ eglot--managed-mode
>>>      (unless (eglot--stay-out-of-p 'imenu)
>>>        (add-function :before-until (local 'imenu-create-index-function)
>>>                      #'eglot-imenu))
>>> -    (unless (eglot--stay-out-of-p 'flymake) (flymake-mode 1))
>>> +    (unless (eglot--stay-out-of-p 'flymake)
>>> +      (if flymake-mode (flymake-start) (flymake-mode 1)))
>>>      (unless (eglot--stay-out-of-p 'eldoc)
>>>        (add-hook 'eldoc-documentation-functions #'eglot-hover-eldoc-function
>>>                  nil t)
>>
>> Spencer, any comments?
>>
>> From where I stand, this is okay for the emacs-30 release branch,
>> unless you think it could break some legitimate workflow.
>
> Yes, this seems good for emacs-30.  Thanks Eshel!

Great, thanks.  Since this is a change in eglot.el, let me also ask João
before installing: João, any objections to the change above?


Cheers,

Eshel

Previous Next

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