Michael Albinus <michael.albinus@gmx.de> writes:

> João Távora <joaotavora@gmail.com> writes:
>
> Hi João,
>
>> Michael, now you've brought back the Eglot/Tramp hang of bug#61350!
>>
>> Really Michael, please consider reverting 0330cff65ae (your latest) and
>> 54ef338ba36 (from two days ago), and going back to the simpler version
>> that you originally proposed, with just the 'while' added there.
>
> Honestly, I'm lost with all the patches back forth and back. Could you
> pls show me the change you propose, based on the current master state? 

OK.  Based on the current master state, I'm sending you 4 (four
patches).  I known it looks a lot, but is quite simple.

'0001-Revert-Michael-Albinus-Fix-regression-in-Tramp-bug-6.patch'

   Reverts your latest commit 0330cff65ae, where some settings of
   'shared-socket' are commented out.

'0002-Revert-Michael-Albinus-Improve-Tramp-processes-to-ac.patch'

   Reverts 54ef338ba36 from two days ago, for all purposes restoring the
   state of Tramp to what is in emacs-29.

'0003-A-simpler-fix-for-bug-61350-a-small-tweak-Michael-s-.patch'

   This is the simpler patch that adds the (while
   (accept-process-output)).  It is directly based on your first idea.
   It keeps the timeout.  It is the most conservative patch.

'0004-Just-for-testing-Remove-Tramp-specific-workaround-in.patch'

   This removes the Tramp-specific Eglot workaround.

   It's good for testing the original hang reporting in bug#61350

After you have applied these four patches, the following should happen:

* Tramp tests should have the same pass rate in master as in emacs-29

* bug#61350 should be fixed -- even without the Eglot workaround.  Let
  me know if you need help testing this (I have a nifty dockerfile and
  some command-line incantations that make testing very easy).

João