Re: Tramp and NT-Emacs with plink

[Kai Großjohann]

Douglas Gray Stephens <address@hidden> writes:

> I had not upgraded to the current version till just now, so I was
> running 2.0.19 with your last patch (adding an extract part to the AND
> statement after
>                (> (tramp-time-diff (current-time) tramp-last-cmd-time) 60))
> on line 5299).
>
> Anyway doing a find file
>   /plink:address@hidden:public_html/test.html
> I noticed that I was getting prompted for a password twice (again an  
>  echo are you awake
> statement was triggering a second prompt for a password. An extract of
> the debug buffer which continues from a session last used 24+ hours
> ago is appended. 

Wow, your debugging help is invaluable.  Thanks a lot.  Please try
this patch (or the current CVS):

cvs server: Diffing .
cvs server: Diffing contrib
cvs server: Diffing lisp
Index: lisp/ChangeLog
===================================================================
RCS file: /cvsroot/tramp/tramp/lisp/ChangeLog,v
retrieving revision 2.223
diff -u -r2.223 ChangeLog
--- lisp/ChangeLog      27 Sep 2002 21:37:15 -0000      2.223
+++ lisp/ChangeLog      28 Sep 2002 11:52:09 -0000
@@ -1,3 +1,11 @@
+2002-09-28  Kai Großjohann  <address@hidden>
+
+       * tramp.el (tramp-open-connection-setup-interactive-shell):
+       Explicitly set tramp-last-cmd-time before invoking
+       tramp-send-command the first time.  Otherwise, tramp-send-command
+       would issue "echo are you awake" right at the first time, which
+       is not what we want.  Reported by Douglas Gray Stephens.
+
 2002-09-27  Michael Albinus  <address@hidden>
 
        * tramp.el (tramp-methods, tramp-completion-function-alist):
Index: lisp/tramp.el
===================================================================
RCS file: /cvsroot/tramp/tramp/lisp/tramp.el,v
retrieving revision 2.231
diff -u -r2.231 tramp.el
--- lisp/tramp.el       27 Sep 2002 21:37:15 -0000      2.231
+++ lisp/tramp.el       28 Sep 2002 11:52:10 -0000
@@ -5033,7 +5033,10 @@
   (erase-buffer)
   (tramp-message 9 "Setting shell prompt")
   ;; Douglas Gray Stephens <address@hidden> says that we must
-  ;; use "\n" here, not tramp-rsh-end-of-line.
+  ;; use "\n" here, not tramp-rsh-end-of-line.  We also manually frob
+  ;; the last time we sent a command, to avoid tramp-send-command to send
+  ;; "echo are you awake".
+  (setq tramp-last-cmd-time (current-time))
   (tramp-send-command
    multi-method method user host
    (format "PS1='%s%s%s'; PS2=''; PS3=''"
cvs server: Diffing test
cvs server: Diffing texi
cvs server: Diffing tramp2


> I have just upgraded to 2.023, and your latest patch may have
> addressed this issue.

Probably not.

> For some reason cvs on my linux box failed to pull down the changes
> (using
>  cvs -z3 -d:pserver:address@hidden:/cvsroot/tramp update tramp
> ) so I went to the WEB page, thinking I would browse the source for
> changes, anyway, 
>  http://www.nongnu.org/tramp/#Obtaining%20%3csmall%3eTRAMP%3c%2fsmall%3e 
> could do with an update (I'm not sure if you take care of the page).
> There are some examples for using CVS
>
>   Or follow the example session below: 
>
>   ] cd ~/emacs
>   ] cvs -d:pserver:address@hidden:/cvsroot/tramp login
>
>   (Logging in to address@hidden)
>   CVS password: (just hit RET here)
>   ...
>
>   ] cvs -z3 -d:pserver:address@hidden:/cvsroot/tramp co tramp
>
>   You should now have a directory ~/emacs/tramp containing the latest version 
> of TRAMP. You can fetch the latest updates from the
>   repository by issuing the command: 
>
>   ] cd ~/emacs/tramp
>   ] cvs update -d
>
> Personally I've been using 
>  cd ~/emacs
>  cvs -z3 -d:pserver:address@hidden:/cvsroot/tramp update tramp
> (on Windows or Linux) as I do not have the CVSROOT environment
> variable set.
>
> I would suggest that the page is updated to either get people to set
> the CVSROOT, or to specify the root on the command for update.

It is only necessary to specify the repository on the first checkout
(which needs to be done using co instead of update).

Note that the manual says to cd to the ~/emacs/tramp directory.  There, you
(and CVS) can find the CVSROOT value to use in the file CVS/Root.  If
you want to make an update from the ~/emacs directory, then indeed
you need to specify the CVSROOT value on the command line.  But I
think switching to the ~/emacs/tramp directory and then just doing
"cvs up -dP" is more convenient.

kai
-- 
~/.signature is: umop ap!sdn    (Frank Nobis)