From: Michael Albinus <michael.albinus@gmx.de>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: bob@rattlesnake.com, emacs-devel@gnu.org
Subject: Re: Emacs tramp troubles with old Sun
Date: Mon, 12 Nov 2007 21:23:35 +0100 [thread overview]
Message-ID: <87mytjusag.fsf@gmx.de> (raw)
In-Reply-To: <nqmytokbjg.fsf@alcatel-lucent.de> (Michael Albinus's message of "Thu, 08 Nov 2007 16:21:55 +0100")
I wrote:
> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
>> Would it be possible for Tramp to detect the "test: argument expected" error
>> message and force a "cleanup-connection" at that point?
>> [ and of course, do similar things wherever applicable: the idea is to try
>> and detect when the cached properties become invalid, which will
>> necessarily happen over time as machines get upgraded/replaced/... ]
>
> Exactly the same idea I'm just thinking about. But one must ensure
> there's no infloop - when "test" does not work, it could be another
> reason but invalid cached values. One shall remember the reason for
> calling cleanup-connection, and when the same problem appears again,
> it's time to give up.
>
> Another idea is to keep a counter for cached properties. After a while
> (100 uses, 100 days, whatever) the values shall be recomputed, just in
> case. Think about a changed default $PATH, where it isn't always an
> obvious error with the old value, when new directories appear.
>
> At least when "uname -s -r" returns a changed value, it is time to
> expire the cached properties for that machine.
Finally, I've implemented only the "uname -sr" check. The other checks
for connection properties cleanups I have had in mind are too
dangerous. Tramp sends remote commands on the fly (when it is necessary
only). Therefore, command sequences are not "atomic" such a way one
could just flush the cache, and reapply the command.
Another drawback would be that error analysis might be impossible when
all relevant data are thrown away once an error happened.
The original problem we have spoken about was that Tramp did not
recognize shell echoing on the "old Sun". Wrong cache entries were
likely a result of _this_ error. I expect to reduce the number of errors
in Tramp :-) Such situations should not happen so often in the future.
>> -- Stefan
Best regards, Michael.
prev parent reply other threads:[~2007-11-12 20:23 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-06 14:11 Emacs tramp troubles with old Sun Robert J. Chassell
2007-11-06 19:38 ` Michael Albinus
2007-11-07 1:24 ` Robert J. Chassell
2007-11-07 7:37 ` Michael Albinus
2007-11-07 10:08 ` Andreas Schwab
2007-11-07 10:30 ` Michael Albinus
2007-11-07 20:50 ` Michael Albinus
2007-11-07 21:27 ` Robert J. Chassell
2007-11-08 6:26 ` Michael Albinus
2007-11-08 11:12 ` Robert J. Chassell
2007-11-08 14:52 ` Stefan Monnier
2007-11-08 15:21 ` Michael Albinus
2007-11-12 20:23 ` Michael Albinus [this message]
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=87mytjusag.fsf@gmx.de \
--to=michael.albinus@gmx.de \
--cc=bob@rattlesnake.com \
--cc=emacs-devel@gnu.org \
--cc=monnier@iro.umontreal.ca \
/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.