* bug#2410: 23.0.90; tramp: spurious ftp-error
@ 2009-02-21 15:02 Chong Yidong
2009-02-21 20:09 ` Stephen Berman
2009-02-22 16:05 ` Juri Linkov
0 siblings, 2 replies; 9+ messages in thread
From: Chong Yidong @ 2009-02-21 15:02 UTC (permalink / raw)
To: Stephen Berman; +Cc: 2410
> When I use Tramp to do an ftp transfer from Dired, I get error messages
> like this:
>
> Copy `/local/path/of/source/file' to
> `/ftp:name@server.dot.com:/remote/path/of/destination/file' failed:
> (ftp-error FTP Error: DIR failed: usage: ls [remote-path [local-file]])
>
> However, the file transer was in fact successful. This started
> happening some time within the last two or three weeks, I believe,
> though I cannot date it precisely.
I can't reproduce this easily, since I don't have access to an FTP
server for testing. Could you try bisecting prior revisions to find
when this bug first appeared?
Thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2410: 23.0.90; tramp: spurious ftp-error 2009-02-21 15:02 bug#2410: 23.0.90; tramp: spurious ftp-error Chong Yidong @ 2009-02-21 20:09 ` Stephen Berman 2009-02-22 10:29 ` Michael Albinus 2009-02-22 16:05 ` Juri Linkov 1 sibling, 1 reply; 9+ messages in thread From: Stephen Berman @ 2009-02-21 20:09 UTC (permalink / raw) To: Chong Yidong; +Cc: 2410 On Sat, 21 Feb 2009 10:02:23 -0500 Chong Yidong <cyd@stupidchicken.com> wrote: >> When I use Tramp to do an ftp transfer from Dired, I get error messages >> like this: >> >> Copy `/local/path/of/source/file' to >> `/ftp:name@server.dot.com:/remote/path/of/destination/file' failed: >> (ftp-error FTP Error: DIR failed: usage: ls [remote-path [local-file]]) >> >> However, the file transer was in fact successful. This started >> happening some time within the last two or three weeks, I believe, >> though I cannot date it precisely. > > I can't reproduce this easily, since I don't have access to an FTP > server for testing. Could you try bisecting prior revisions to find > when this bug first appeared? > > Thanks. I fear I won't have time to do this very soon, but if I find the time, I'll report back with the results. Steve Berman ^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2410: 23.0.90; tramp: spurious ftp-error 2009-02-21 20:09 ` Stephen Berman @ 2009-02-22 10:29 ` Michael Albinus 0 siblings, 0 replies; 9+ messages in thread From: Michael Albinus @ 2009-02-22 10:29 UTC (permalink / raw) To: Stephen Berman; +Cc: Chong Yidong, 2410 Stephen Berman <stephen.berman@gmx.net> writes: > On Sat, 21 Feb 2009 10:02:23 -0500 Chong Yidong <cyd@stupidchicken.com> wrote: > >>> When I use Tramp to do an ftp transfer from Dired, I get error messages >>> like this: >>> >>> Copy `/local/path/of/source/file' to >>> `/ftp:name@server.dot.com:/remote/path/of/destination/file' failed: >>> (ftp-error FTP Error: DIR failed: usage: ls [remote-path [local-file]]) >>> >>> However, the file transer was in fact successful. This started >>> happening some time within the last two or three weeks, I believe, >>> though I cannot date it precisely. >> >> I can't reproduce this easily, since I don't have access to an FTP >> server for testing. Could you try bisecting prior revisions to find >> when this bug first appeared? >> >> Thanks. > > I fear I won't have time to do this very soon, but if I find the time, > I'll report back with the results. Maybe you could show us a backtrace in case of error as well as the ange-ftp buffer? The last changes to ange-ftp are trivial ones, which don't look suspicious: RCS file: /sources/emacs/emacs/lisp/net/ange-ftp.el,v Working file: ange-ftp.el head: 1.102 branch: locks: strict access list: keyword substitution: kv total revisions: 195; selected revisions: 195 description: ---------------------------- revision 1.102 date: 2009-01-05 04:22:43 +0100; author: gm; state: Exp; lines: +1 -1; commitid: qZ3e2OYSprEA3dxt; Add 2009 to copyright years. ---------------------------- revision 1.101 date: 2008-11-16 06:50:26 +0100; author: lektu; state: Exp; lines: +202 -219; commitid: kOd4tyyfYMkJwNqt; * net/ange-ftp.el (ange-ftp-multi-msgs, ange-ftp-good-msgs) (ange-ftp-try-passive-mode, ange-ftp-data-buffer-name) (ange-ftp-account-hashtable, ange-ftp-ls-cache-lsargs) (ange-ftp-ls-cache-file, ange-ftp-ls-cache-res, ange-ftp-get-user) (ange-ftp-ftp-name-component, ange-ftp-kill-ftp-process) (ange-ftp-quote-string, ange-ftp-process-handle-line) (ange-ftp-start-process, ange-ftp-send-cmd, ange-ftp-add-dumb-unix-host) (ange-ftp-before-parse-ls-hook, ange-ftp-after-parse-ls-hook) (ange-ftp-ls, ange-ftp-add-dl-dir, ange-ftp-get-file-entry) (ange-ftp-set-binary-mode, ange-ftp-set-ascii-mode, ange-ftp-get-pwd) (ange-ftp-file-name-as-directory-alist, ange-ftp-reread-dir) (ange-ftp-vms-filename-regexp, ange-ftp-bs2000-fix-name-regexp-reverse) (ange-ftp-bs2000-fix-name-regexp): Fix typos in docstrings. (ange-ftp-name-format, ange-ftp-gateway-fatal-msgs) (ange-ftp-xfer-size-msgs, ange-ftp-tmp-name-template) (ange-ftp-netrc-filename, ange-ftp-disable-netrc-security-check) (ange-ftp-default-user, ange-ftp-default-password) (ange-ftp-default-account, ange-ftp-netrc-default-password) (ange-ftp-netrc-default-account, ange-ftp-dumb-unix-host-regexp) (ange-ftp-binary-file-name-regexp, ange-ftp-gateway-host) (ange-ftp-gateway-prompt-pattern, ange-ftp-smart-gateway-port) (ange-ftp-send-hash, ange-ftp-binary-hash-mark-size) (ange-ftp-ascii-hash-mark-size, ange-ftp-process-verbose) (ange-ftp-ftp-program-name, ange-ftp-gateway-ftp-program-name) (ange-ftp-ftp-program-args, ange-ftp-nslookup-program) (ange-ftp-make-backup-files, ange-ftp-retry-time) (ange-ftp-bs2000-special-prefix): Remove * from defcustom docstrings. (ange-ftp-skip-msgs, ange-ftp-potential-error-msgs) (ange-ftp-gateway-tmp-name-template) (ange-ftp-generate-anonymous-password, ange-ftp-local-host-regexp) (ange-ftp-gateway-program-interactive, ange-ftp-smart-gateway) (ange-ftp-raw-login): Remove * from defcustom docstrings; fix typos. (ange-ftp-fatal-msgs): Remove * from defcustom docstring; doc fix. (ange-ftp-gateway-program): Remove * from docstring and reflow. (ange-ftp-hash-entry-exists-p, ange-ftp-hash-table-keys) (ange-ftp-raw-send-cmd, ange-ftp-get-files, ange-ftp-canonize-filename) (ange-ftp-file-name-as-directory, ange-ftp-directory-file-name): (ange-ftp-copy-files-async, ange-ftp-rename-remote-to-remote): (ange-ftp-rename-local-to-remote): Doc fixes. (ange-ftp-set-xfer-size, ange-ftp-call-cont, ange-ftp-process-filter): Use `when', `unless'. (ange-ftp-set-passwd): Rename arg PASSWD to PASSWORD. (ange-ftp-process-handle-hash): Rename arg STR to STRING. (ange-ftp-nslookup-host): Rename arg HOST to HOSTNAME. (ange-ftp-smart-login): Rename arg PASS to PASSWORD. (ange-ftp-normal-login): Rename arg PASS to PASSWORD. Fix typo. (ange-ftp-process-sentinel): Use `when'. Fix typo. (ange-ftp-gwp-start): Use `let', not `let*'; use `when'. Fix typo. (ange-ftp-fix-name-func-alist, ange-ftp-fix-dir-name-func-alist) (ange-ftp-parse-list-func-alist, ange-ftp-add-file-entry-alist) (ange-ftp-delete-file-entry-alist): Fix typos and reflow docstring. (ange-ftp-dumb-unix-host, ange-ftp-binary-file) (ange-ftp-directory-files, ange-ftp-file-modtime, ange-ftp-vms-host) (ange-ftp-mts-host, ange-ftp-cms-host, ange-ftp-bs2000-host) (ange-ftp-bs2000-posix-host): Use `string-match-p' instead of `(save-match-data (string-match ...))'. (ange-ftp-use-gateway-p, ange-ftp-use-smart-gateway-p) (ange-ftp-file-name-directory, ange-ftp-file-name-nondirectory): Use `string-match-p' instead of `(save-match-data (string-match ...))'. Doc fixes. ---------------------------- revision 1.100 date: 2008-09-29 07:41:56 +0200; author: albinus; state: Exp; lines: +3 -1; commitid: NynrHl6ahJBL1Dkt; * net/ange-ftp.el (ange-ftp-file-remote-p): Handle `localname' as identification. > Steve Berman Best regards, Michael. ^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2410: 23.0.90; tramp: spurious ftp-error 2009-02-21 15:02 bug#2410: 23.0.90; tramp: spurious ftp-error Chong Yidong 2009-02-21 20:09 ` Stephen Berman @ 2009-02-22 16:05 ` Juri Linkov 2009-02-22 16:27 ` Juri Linkov 1 sibling, 1 reply; 9+ messages in thread From: Juri Linkov @ 2009-02-22 16:05 UTC (permalink / raw) To: Chong Yidong; +Cc: Stephen Berman, 2410 >> When I use Tramp to do an ftp transfer from Dired, I get error messages >> like this: >> >> Copy `/local/path/of/source/file' to >> `/ftp:name@server.dot.com:/remote/path/of/destination/file' failed: >> (ftp-error FTP Error: DIR failed: usage: ls [remote-path [local-file]]) >> >> However, the file transer was in fact successful. This started >> happening some time within the last two or three weeks, I believe, >> though I cannot date it precisely. > > I can't reproduce this easily, since I don't have access to an FTP > server for testing. Could you try bisecting prior revisions to find > when this bug first appeared? I can reproduce it. It can be narrowed down to 2009-02-11 change: * dired-aux.el (dired-add-entry): Avoid munging long-form Dired switches (Bug#2261). -- Juri Linkov http://www.jurta.org/emacs/ ^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2410: 23.0.90; tramp: spurious ftp-error 2009-02-22 16:05 ` Juri Linkov @ 2009-02-22 16:27 ` Juri Linkov 2009-02-22 18:19 ` Chong Yidong 0 siblings, 1 reply; 9+ messages in thread From: Juri Linkov @ 2009-02-22 16:27 UTC (permalink / raw) To: 2410; +Cc: Chong Yidong, Stephen Berman > I can reproduce it. It can be narrowed down to 2009-02-11 change: > > * dired-aux.el (dired-add-entry): Avoid munging long-form Dired > switches (Bug#2261). It conflicts with code that removes the final "d" from switches in `ange-ftp-insert-directory'. When switches are "--dired -ald" then the FTP command is correct: "ls -al filename". When switches are "--dired -al -d" then the FTP command is incorrect: "ls -al - filename". -- Juri Linkov http://www.jurta.org/emacs/ ^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2410: 23.0.90; tramp: spurious ftp-error 2009-02-22 16:27 ` Juri Linkov @ 2009-02-22 18:19 ` Chong Yidong 2009-02-22 19:04 ` Juri Linkov 0 siblings, 1 reply; 9+ messages in thread From: Chong Yidong @ 2009-02-22 18:19 UTC (permalink / raw) To: Juri Linkov; +Cc: Stephen Berman, 2410 Juri Linkov <juri@jurta.org> writes: >> I can reproduce it. It can be narrowed down to 2009-02-11 change: >> >> * dired-aux.el (dired-add-entry): Avoid munging long-form Dired >> switches (Bug#2261). > > It conflicts with code that removes the final "d" from switches > in `ange-ftp-insert-directory'. Does this fix it? *** trunk/lisp/net/ange-ftp.el.~1.102.~ 2009-01-06 15:29:22.000000000 -0500 --- trunk/lisp/net/ange-ftp.el 2009-02-22 13:16:25.000000000 -0500 *************** *** 4478,4484 **** ;; then do an ls of current dir, which obviously won't work if we ;; want to ls a file. So instead, we get a full listing of the ;; parent directory and extract the line corresponding to `file'. ! (when (string-match "d\\'" switches) ;; Remove "d" which dired added to `switches'. (setq switches (substring switches 0 (match-beginning 0)))) (let* ((dirlist (ange-ftp-ls (or (file-name-directory file) ".") --- 4478,4484 ---- ;; then do an ls of current dir, which obviously won't work if we ;; want to ls a file. So instead, we get a full listing of the ;; parent directory and extract the line corresponding to `file'. ! (when (string-match "-?d\\'" switches) ;; Remove "d" which dired added to `switches'. (setq switches (substring switches 0 (match-beginning 0)))) (let* ((dirlist (ange-ftp-ls (or (file-name-directory file) ".") ^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2410: 23.0.90; tramp: spurious ftp-error 2009-02-22 18:19 ` Chong Yidong @ 2009-02-22 19:04 ` Juri Linkov 2009-02-24 0:00 ` Stephen Berman 0 siblings, 1 reply; 9+ messages in thread From: Juri Linkov @ 2009-02-22 19:04 UTC (permalink / raw) To: Chong Yidong; +Cc: Stephen Berman, 2410 > Does this fix it? > > *** trunk/lisp/net/ange-ftp.el.~1.102.~ 2009-01-06 15:29:22.000000000 -0500 > --- trunk/lisp/net/ange-ftp.el 2009-02-22 13:16:25.000000000 -0500 > *************** > *** 4478,4484 **** > ;; then do an ls of current dir, which obviously won't work if we > ;; want to ls a file. So instead, we get a full listing of the > ;; parent directory and extract the line corresponding to `file'. > ! (when (string-match "d\\'" switches) > ;; Remove "d" which dired added to `switches'. > (setq switches (substring switches 0 (match-beginning 0)))) > (let* ((dirlist (ange-ftp-ls (or (file-name-directory file) ".") > --- 4478,4484 ---- > ;; then do an ls of current dir, which obviously won't work if we > ;; want to ls a file. So instead, we get a full listing of the > ;; parent directory and extract the line corresponding to `file'. > ! (when (string-match "-?d\\'" switches) > ;; Remove "d" which dired added to `switches'. > (setq switches (substring switches 0 (match-beginning 0)))) > (let* ((dirlist (ange-ftp-ls (or (file-name-directory file) ".") Thanks, it works. I'd also ask Stephen to confirm this fixes his original bug report. -- Juri Linkov http://www.jurta.org/emacs/ ^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2410: 23.0.90; tramp: spurious ftp-error 2009-02-22 19:04 ` Juri Linkov @ 2009-02-24 0:00 ` Stephen Berman 0 siblings, 0 replies; 9+ messages in thread From: Stephen Berman @ 2009-02-24 0:00 UTC (permalink / raw) To: Juri Linkov; +Cc: Chong Yidong, 2410 On Sun, 22 Feb 2009 21:04:14 +0200 Juri Linkov <juri@jurta.org> wrote: >> Does this fix it? >> >> *** trunk/lisp/net/ange-ftp.el.~1.102.~ 2009-01-06 15:29:22.000000000 -0500 >> --- trunk/lisp/net/ange-ftp.el 2009-02-22 13:16:25.000000000 -0500 >> *************** >> *** 4478,4484 **** >> ;; then do an ls of current dir, which obviously won't work if we >> ;; want to ls a file. So instead, we get a full listing of the >> ;; parent directory and extract the line corresponding to `file'. >> ! (when (string-match "d\\'" switches) >> ;; Remove "d" which dired added to `switches'. >> (setq switches (substring switches 0 (match-beginning 0)))) >> (let* ((dirlist (ange-ftp-ls (or (file-name-directory file) ".") >> --- 4478,4484 ---- >> ;; then do an ls of current dir, which obviously won't work if we >> ;; want to ls a file. So instead, we get a full listing of the >> ;; parent directory and extract the line corresponding to `file'. >> ! (when (string-match "-?d\\'" switches) >> ;; Remove "d" which dired added to `switches'. >> (setq switches (substring switches 0 (match-beginning 0)))) >> (let* ((dirlist (ange-ftp-ls (or (file-name-directory file) ".") > > Thanks, it works. I'd also ask Stephen to confirm this fixes his > original bug report. Sorry for the belated response, but I can indeed confirm that this eliminates the spurious ftp-error message. Many thanks for quickly finding the source of the problem! Steve Berman ^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2410: 23.0.90; tramp: spurious ftp-error @ 2009-02-20 16:20 Stephen Berman 0 siblings, 0 replies; 9+ messages in thread From: Stephen Berman @ 2009-02-20 16:20 UTC (permalink / raw) To: emacs-pretest-bug When I use Tramp to do an ftp transfer from Dired, I get error messages like this: Copy `/local/path/of/source/file' to `/ftp:name@server.dot.com:/remote/path/of/destination/file' failed: (ftp-error FTP Error: DIR failed: usage: ls [remote-path [local-file]]) However, the file transer was in fact successful. This started happening some time within the last two or three weeks, I believe, though I cannot date it precisely. In GNU Emacs 23.0.90.4 (i686-pc-linux-gnu, GTK+ Version 2.14.4) of 2009-02-20 on escher Windowing system distributor `The X.Org Foundation', version 11.0.10502000 Important settings: value of $LC_ALL: nil value of $LC_COLLATE: nil value of $LC_CTYPE: nil value of $LC_MESSAGES: nil value of $LC_MONETARY: nil value of $LC_NUMERIC: nil value of $LC_TIME: nil value of $LANG: en_US.UTF-8 value of $XMODIFIERS: @im=local locale-coding-system: utf-8-unix default-enable-multibyte-characters: t ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2009-02-24 0:00 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2009-02-21 15:02 bug#2410: 23.0.90; tramp: spurious ftp-error Chong Yidong 2009-02-21 20:09 ` Stephen Berman 2009-02-22 10:29 ` Michael Albinus 2009-02-22 16:05 ` Juri Linkov 2009-02-22 16:27 ` Juri Linkov 2009-02-22 18:19 ` Chong Yidong 2009-02-22 19:04 ` Juri Linkov 2009-02-24 0:00 ` Stephen Berman -- strict thread matches above, loose matches on Subject: below -- 2009-02-20 16:20 Stephen Berman
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.