From: Eshel Yaron via "Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Thierry Volpiatto <thievol@posteo.net>,
kun.liu@gmail.com, 67702@debbugs.gnu.org
Subject: bug#67702: 30.0.50; insert-register can no longer be used in minibuffer
Date: Fri, 08 Dec 2023 08:31:15 +0100 [thread overview]
Message-ID: <m1plzhgngc.fsf@dazzs-mbp.home> (raw)
In-Reply-To: <83bkb117y2.fsf@gnu.org> (Eli Zaretskii's message of "Fri, 08 Dec 2023 09:15:17 +0200")
Hi Eli,
Eli Zaretskii <eliz@gnu.org> writes:
>> From: Thierry Volpiatto <thievol@posteo.net>
>> Cc: Kun Liu <kun.liu@gmail.com>, 67702@debbugs.gnu.org
>> Date: Fri, 08 Dec 2023 07:04:53 +0000
>>
>> >> 1) Save some text in register a
>> >> 2) M-x query-replace
>> >> 3) in the minibuffer, type C-x r i
>> >> 4) got error: "byte-code: Command attempted to use minibuffer while in minibuffer"
Yes, this regression is one of the issues I pointed at with the recent
`register-read-with-preview` changes.
>> > I guess register-read-with-preview should temporarily bind
>> > enable-recursive-minibuffers to a non-nil value?
>>
>> Yes, do you want me to install this change?
>
> If you think that's the correct solution, sure.
FWIW, I think it's not the right solution. As I wrote in bug#66394, I
think it's wrong to involve the minibuffer in reading registers in any
way. `enable-recursive-minibuffers` would make this less broken, but
only slightly.
>> I have also a pending patch to apply for bug#66394 (see it there).
>
> You said you will install that if no one objects. I'm not sure what
> was the conclusion of the discussion with Eshel about that.
It's up to you maintainers to decide, I think. Following your request,
I've proposed a patch that reverts Thierry's changes, and implements the
parts I find useful in a clean and backward compatible way.
> If it's unrelated, then please go ahead and install your changes in
> that discussion. In any case, perhaps you could help Eshel improve
> and polish his additions, which AFAIU are supposed to provide an
> optional behavior more similar to the previous one.
That'd be nice, thanks.
Best,
Eshel
next prev parent reply other threads:[~2023-12-08 7:31 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-07 22:33 bug#67702: 30.0.50; insert-register can no longer be used in minibuffer Kun Liu
2023-12-08 6:32 ` Eli Zaretskii
2023-12-08 7:04 ` Thierry Volpiatto
2023-12-08 7:14 ` Thierry Volpiatto
2023-12-08 7:15 ` Eli Zaretskii
2023-12-08 7:31 ` Eshel Yaron via Bug reports for GNU Emacs, the Swiss army knife of text editors [this message]
2023-12-08 7:52 ` Eli Zaretskii
2023-12-08 8:27 ` Eshel Yaron via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-12-08 8:51 ` Eli Zaretskii
2023-12-08 9:07 ` Eshel Yaron via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-12-08 10:23 ` Thierry Volpiatto
2023-12-08 10:16 ` Thierry Volpiatto
2023-12-08 10:40 ` Eshel Yaron via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-12-08 12:00 ` Eli Zaretskii
2023-12-08 12:11 ` Thierry Volpiatto
2023-12-08 12:39 ` Thierry Volpiatto
2023-12-08 12:40 ` Thierry Volpiatto
2023-12-08 16:37 ` Thierry Volpiatto
2023-12-08 8:46 ` Thierry Volpiatto
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m1plzhgngc.fsf@dazzs-mbp.home \
--to=bug-gnu-emacs@gnu.org \
--cc=67702@debbugs.gnu.org \
--cc=eliz@gnu.org \
--cc=kun.liu@gmail.com \
--cc=me@eshelyaron.com \
--cc=thievol@posteo.net \
/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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.