bug#39687: 26.3; Add customize-variable option for not locking keyboards

[Logan Perkins]

On 2/22/20 1:27 AM, Eli Zaretskii wrote:
> [Please keep the bug address on the CC list, so this whole discussion
> is recorded by the Emacs issue tracker.]

Oops! I hit reply instead of reply-all, not sure why Thunderbird
doesn't make that the default, but I'll be more mindful of that in the 
future.

>
>> From: Logan Perkins <logan@lp-programming.com>
>> Date: Fri, 21 Feb 2020 10:37:39 -0800
>>
>> On 2/21/20 12:23 AM, Eli Zaretskii wrote:
>> >> From: Logan Perkins <logan@lp-programming.com>
>> >> Date: Wed, 19 Feb 2020 21:01:30 -0800
>> >>
>> >> Is there some further reason to lock the keyboard that I haven't
>> >> considered?
>> >
>> > Can we back up a little, and discuss the use cases where the current
>> > behavior presents a limitation? Is quitting in the other clients the
>> > only one, or are there more?
>>
>> Quitting in other clients is one, but fairly minor (C-z; kill %1 will
>> get you out of it). Switching clients generally is another minor case.
>> If you walk away with the minibuffer open by accident, and then try to
>> use a remote client (via SSH or similar) later, it's locked (you can
>> work around this by registering a SIGUSR handler to close the
>> minibuffer, but that's not ideal).
>
> These seem to be valid use cases, so I tend to agree we should have an
> easier way of breaking out of the minibuffer input in another client.
>
>> > Also, are you implicitly saying that several persons work
>> > simultaneously vis-�-vis the same Emacs server? Because if not, I'm
>> > not sure I understand how simultaneous need to input from different
>> > clients could even happen.
>>
>> That's exactly the use-case where it matters most. If you're familiar
>> with Ludum Dare and similar code-sprints, it's pretty common to
>> have multiple people working on the same files at the same time. Having
>> a shared editor makes it faster and easier to draw attention to exactly
>> where one person needs help. It's also great for teaching (when you
>> aren't physically in front of the same computer), or for onboarding new
>> team members. Screen (the terminal multiplexer) can be used to similar
>> effect, but the ability to simultaneously edit the *same* file is
>> specific to emacs.
>
> I don't understand what you expect Emacs to do in these use cases. If
> we process inputs from several clients as they arrive, we could
> produce results that are unexpected and even disastrous. For example,
> suppose we receive C-x from one client followed by C-u from another
> followed by C-s from the first one -- if we process these in the order
> they were received, the result will be none of what the two clients
> intended.
>
> Maybe you thought that our input code will process input in chunks of
> complete sequences, and thus avoid the above-mentioned disasters, but
> then (a) I think we will need a very thorough restructuring of the
> current code in keyboard.c, as it currently decides on this
> dynamically; and (b) you will still have the same problem if the user
> of one client types C-x and then pauses for some reason.
>
> So I'm afraid I don't see what kind of solution is sought for here,
> please clarify.


So I've just done some more testing, with emacs 26.3-r1 (the latest
stable version in Gentoo), and emacs 27.0.50_pre20191223, the latest
snapshot, both compiled with and without the gutted
switch_to_single_kboard... (I'll see about getting the latest
development version from the repo probably this evening).

Looks like keyboard handling got changed sometime between the two
versions. I ran the following sequence in all 4 copies.

1. ./emacs -nw #on seat0
2. M-x start-server
3. ./emacsclient -t #on seat1
4. Switch both to the scratch buffer
5. Put each seat's cursor on its own line
6. On seat0, type abcde
7. On seat1, type 12345
8. On seat0, type C-x
9. On seat1, type u
10. On seat1, type C-x
11. On seat0, type C-c

Note that this sequence doesn't try to put emacs into single keyboard
mode, so the gutted function has no impact on the results.

In emacs-26, step 9 inserts a literal u on seat1's line, step 11 closes
emacs. Before step 11, both seat and seat1 display C-x as "breadcrumbs"
in the minibuffer.

In emacs-27, step 9 undoes seat0's last action (removes abcde), step 11
closes seat1's emacsclient.

Obviously, the behaviour in emacs-27 precludes simultanuous input. I
also think it's poor behavior, even if we don't unlock the keyboard when
the minibuffer is in use, since someone walking away after hitting C-x
(or some other partial command) and then connecting later will get
unexpected (and probably unseen) results with their first keypress.

I'll see if I can figure out which changes in keyboard.c account for the
changed behavior, and what the reason for them was.