all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Dmitry Gutov <dmitry@gutov.dev>
To: Michael Albinus <michael.albinus@gmx.de>
Cc: 70901@debbugs.gnu.org
Subject: bug#70901: 30.0.50; Tramp doesn't use ControlMaster even with (setq tramp-use-connection-share nil)
Date: Sat, 18 May 2024 17:11:44 +0300	[thread overview]
Message-ID: <440e9074-a168-4e80-9911-b96f8c20cd60@gutov.dev> (raw)
In-Reply-To: <87le47xt89.fsf@gmx.de>

Hi Michael,

On 18/05/2024 14:19, Michael Albinus wrote:
> Dmitry Gutov <dmitry@gutov.dev> writes:

>> Perhaps it'd make sense to adjust the logging so that the first
>> "...done" is printed right after the connection is established.
> 
> That is already the case. Tramp uses a (slightly modified version) of
> the progress reporter. You'll see two invocations of it, "Opening
> connection ..." and "Waiting for prompts ...". They are cascaded. When
> the second, inner progress reporter finishes, ssh has returned the prompt.

That's really what I meant: when looking at the echo area, I'm given the 
impression that the connection itself takes longer than it really does - 
because the "outer" phase said "Opening connection", and it finishes 
much later than the opening of the connection takes.

> --8<---------------cut here---------------start------------->8---
> 13:01:30.750374 tramp-maybe-open-connection (3) # Opening connection nil for gandalf using ssh...
> 13:01:30.794366 tramp-send-command (6) # exec ssh -o ControlMaster=auto -o ControlPath=/home/albinus/.cache/emacs/tramp.%C -o ControlPersist=no -e none gandalf || exit
> 13:01:30.794769 tramp-process-actions (3) # Waiting for prompts from remote shell...
> 13:01:32.633372 tramp-process-actions (6) #
> Last login: Sat May 18 13:01:31 2024 from 2002:6443:eb5:0:f832:242f:9e14:df3d
> [albinus@gandalf ~]$
> 13:01:32.633453 tramp-process-actions (3) # Waiting for prompts from remote shell...done
> 13:01:35.830072 tramp-maybe-open-connection (3) # Opening connection nil for gandalf using ssh...done
> --8<---------------cut here---------------end--------------->8---
> 
> One modification of Tramp is, that in case of cascaded progress reportes
> only the outer one is visible in the echo area.

That by itself seems reasonable - chattiness in the echo area can be a 
problem by itself.





  reply	other threads:[~2024-05-18 14:11 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-13  2:00 bug#70901: 30.0.50; Tramp doesn't use ControlMaster even with (setq tramp-use-connection-share nil) Dmitry Gutov
2024-05-13  6:07 ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
     [not found]   ` <44185444-8a6a-4924-88f9-853f65a1c61a@gutov.dev>
2024-05-14  9:01     ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-14 19:02       ` Dmitry Gutov
2024-05-15  8:17         ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-15 14:45           ` Dmitry Gutov
2024-05-15 18:15             ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-15 19:55               ` Dmitry Gutov
2024-05-16  9:42                 ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-16 19:50                   ` Dmitry Gutov
2024-05-17 14:57                     ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-18  2:29                       ` Dmitry Gutov
2024-05-18 11:19                         ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-18 14:11                           ` Dmitry Gutov [this message]
2024-05-18 17:00                             ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-18 17:02                               ` Dmitry Gutov
2024-05-18 17:47                                 ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-19  0:07                                   ` Dmitry Gutov
2024-05-23 16:25                                     ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-24 19:12                                       ` Dmitry Gutov
2024-05-25  9:42                                         ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-19  0:49                 ` 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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=440e9074-a168-4e80-9911-b96f8c20cd60@gutov.dev \
    --to=dmitry@gutov.dev \
    --cc=70901@debbugs.gnu.org \
    --cc=michael.albinus@gmx.de \
    /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.