From: "João Távora" <joaotavora@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: dmitry@gutov.dev, 62816@debbugs.gnu.org
Subject: bug#62816: 30.0.50; ElDoc blinks echo area when eldoc-documentation-compose is used
Date: Fri, 14 Apr 2023 20:58:16 +0100 [thread overview]
Message-ID: <CALDnm52jEBufqEzzEy1QqMaLOLKscN-8vTd1mVXPmk+G=WGrFA@mail.gmail.com> (raw)
In-Reply-To: <83wn2ee1mr.fsf@gnu.org>
On Fri, Apr 14, 2023 at 7:53 PM Eli Zaretskii <eliz@gnu.org> wrote:
>
> > Cc: 62816@debbugs.gnu.org
> > From: João Távora <joaotavora@gmail.com>
> > Date: Fri, 14 Apr 2023 19:09:20 +0100
> >
> > On Fri, Apr 14, 2023 at 7:03 PM Dmitry Gutov <dmitry@gutov.dev> wrote:
> > >
> > > On 13/04/2023 17:47, João Távora wrote:
> > > > I'll push it to master soon, but leave this issue open for comments
> > > > and/or feedback a little longer.
> > >
> > > Perhaps we want to backport it to emacs-29 as well? It *is* a bug, after
> > > all.
> >
> > Perhaps. Yes it's a bug. But at this point, I think that's for Eli
> > to decide (i.e. I don't exactly what the policy is now vs two/three
> > weeks ago).
>
> It depends on how bad is the bug, whether it's a regression since
> Emacs 28, and how safe is the fix. In general, bugs should be fixed
> on the release branch, unless the risk associated with the fix is too
> high.
Right, exactly. We were just talking about that. This is not a
regression from Emacs 28 (I'm 99% sure). It was already there.
But the bug is so-so-serious (somewhat annoying flickering, though
not in TTY frames). The fix is simple enough (IMHO, of course,
because I wrote the code ... and the bug).
Do note, also, Eli, that ElDoc is a :core package. So, if we
don't backport, people really really annoyed with this bug (
which manifests itself with Eglot primarily, though it could
manifest with other packages) can get the bugfix if they simply
upgrade ElDoc OR Eglot. Whether it's easy or not to do that
is a matter for bug#62720. Of course they might also get
_other_ bugs and new features, etc if they upgrade.
So maybe that weighs on your decision. You can check out the fix
in 83b5e9cd24ddcbb04dbd5db9a07248ff7fa301ab. If you don't say
anything I will backport in a few days.
João
next prev parent reply other threads:[~2023-04-14 19:58 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-13 14:47 bug#62816: 30.0.50; ElDoc blinks echo area when eldoc-documentation-compose is used João Távora
2023-04-13 22:58 ` Dmitry Gutov
2023-04-13 23:23 ` João Távora
2023-04-13 23:37 ` Dmitry Gutov
2023-04-14 0:16 ` João Távora
2023-04-14 0:22 ` Dmitry Gutov
2023-04-15 1:03 ` Dmitry Gutov
2023-04-14 18:03 ` Dmitry Gutov
2023-04-14 18:09 ` João Távora
2023-04-14 18:53 ` Eli Zaretskii
2023-04-14 19:58 ` João Távora [this message]
2023-04-15 9:13 ` Eli Zaretskii
2023-04-15 11:04 ` João Távora
2023-04-15 11:40 ` Dmitry Gutov
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='CALDnm52jEBufqEzzEy1QqMaLOLKscN-8vTd1mVXPmk+G=WGrFA@mail.gmail.com' \
--to=joaotavora@gmail.com \
--cc=62816@debbugs.gnu.org \
--cc=dmitry@gutov.dev \
--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).