From: Michael Albinus <michael.albinus@gmx.de>
To: Ken Brown <kbrown@cornell.edu>
Cc: 20119@debbugs.gnu.org
Subject: bug#20119: 25.0.50; tramp-test30-special-characters hangs on Cygwin
Date: Wed, 18 Mar 2015 20:53:16 +0100 [thread overview]
Message-ID: <87vbhyt74j.fsf@gmx.de> (raw)
In-Reply-To: <55070931.3080502@cornell.edu> (Ken Brown's message of "Mon, 16 Mar 2015 12:47:45 -0400")
Ken Brown <kbrown@cornell.edu> writes:
> Hi Michael,
Hi Ken,
>> "\t" shall be expanded as <TAB>. And leading (or trailing) spaces are
>> also not allowed in file names on w32; likely it is the same for cygwin.
>
> In the context of file names on Cygwin, "\t" is not expanded to <TAB>
> because of the special treatment of the backslash. But leading and
> trailing spaces are fine; see my patch.
Your patch works fine, but I'm still curious: what happens with this
test, if we don't pass "\t" but the <TAB> character in the file name?
This shall work, shouldn't it?
Maybe you have time to test the follwing patch:
--8<---------------cut here---------------start------------->8---
*** /home/albinus/src/tramp/test/tramp-tests.el.~master~ 2015-03-18 20:50:29.399389219 +0100
--- /home/albinus/src/tramp/test/tramp-tests.el 2015-03-18 16:43:56.073936927 +0100
***************
*** 1522,1528 ****
"Run a simple but comprehensive test over every file in FILES."
(let ((tmp-name1 (tramp--test-make-temp-name))
(tmp-name2 (tramp--test-make-temp-name 'local))
! (files (delq nil files)))
(unwind-protect
(progn
(make-directory tmp-name1)
--- 1522,1529 ----
"Run a simple but comprehensive test over every file in FILES."
(let ((tmp-name1 (tramp--test-make-temp-name))
(tmp-name2 (tramp--test-make-temp-name 'local))
! (files (mapcar (lambda (x) (with-output-to-string (princ x)))
! (delq nil files))))
(unwind-protect
(progn
(make-directory tmp-name1)
***************
*** 1629,1635 ****
(tramp--test-check-files
(if (tramp--test-smb-or-windows-nt-p)
"foo bar baz"
! (if (or (tramp--test-adb-p) (eq system-type 'cygwin))
" foo bar baz "
" foo\tbar baz\t"))
"$foo$bar$$baz$"
--- 1630,1636 ----
(tramp--test-check-files
(if (tramp--test-smb-or-windows-nt-p)
"foo bar baz"
! (if (tramp--test-adb-p)
" foo bar baz "
" foo\tbar baz\t"))
"$foo$bar$$baz$"
--8<---------------cut here---------------end--------------->8---
> Ken
Best regards, Michael.
next prev parent reply other threads:[~2015-03-18 19:53 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-16 16:21 bug#20119: 25.0.50; tramp-test30-special-characters hangs on Cygwin Ken Brown
2015-03-16 16:37 ` Ken Brown
2015-03-16 19:05 ` Michael Albinus
2015-03-16 20:22 ` Ken Brown
2015-03-16 16:39 ` Michael Albinus
2015-03-16 16:47 ` Ken Brown
2015-03-18 19:53 ` Michael Albinus [this message]
2015-03-18 20:10 ` Eli Zaretskii
2015-03-18 21:06 ` Michael Albinus
2015-03-18 21:02 ` Ken Brown
2015-03-18 21:09 ` Michael Albinus
2015-03-16 17:08 ` Eli Zaretskii
2015-03-16 18:56 ` Michael Albinus
2015-03-16 19:32 ` Eli Zaretskii
2015-03-17 11:11 ` Michael Albinus
2015-03-17 12:15 ` Eli Zaretskii
2015-03-17 14:46 ` Michael Albinus
2015-03-17 15:31 ` Eli Zaretskii
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
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87vbhyt74j.fsf@gmx.de \
--to=michael.albinus@gmx.de \
--cc=20119@debbugs.gnu.org \
--cc=kbrown@cornell.edu \
/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 public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).