unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Rasmus <rasmus@gmx.us>
To: michael.albinus@gmx.de
Cc: 19192@debbugs.gnu.org
Subject: bug#19192: 25.0.50; TRAMP uses `dired-listing-switches' for ftp-connection
Date: Thu, 27 Nov 2014 00:45:36 +0100	[thread overview]
Message-ID: <87h9xlil0f.fsf@gmx.us> (raw)
In-Reply-To: <87zjbe9j9c.fsf@gmx.de> (Michael Albinus's message of "Wed, 26 Nov 2014 14:35:59 +0100")

Hi,

Michael Albinus <michael.albinus@gmx.de> writes:

> Rasmus <rasmus@gmx.us> writes:
>> I don't know if this is a feature or a bug.
>>
>> Michael: do you want me to write a patch or will this be a quick fix for
>> you?
>
> Likely, it is a feature. So I don't know whether we can fix this quickly.

I agree.

> However, if you want to investigate it, it is likely in
> ange-ftp.el. Tramp doesn't care about ftp connections, it let
> ange-ftp.el do the job.

Thanks for pointing this out.

> A possible fix might be to give a user an option, whether global
> dired-listing-switches shall be used everywhere, or whether there shall
> be special settings connection-oriented.

So it turns out that arguments are tested via `ange-ftp-switches-ok'.
So the easiest fix would probably be to extend this function to do a
better job of checking.  Unfortunately, I don't know if one can easily
predict which ls commands will be understood and which not?  E.g. does
ftp ls depend on the host systems version of ls?

Thanks,
Rasmus

-- 
Summon the Mothership!







  reply	other threads:[~2014-11-26 23:45 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-26 12:03 bug#19192: 25.0.50; TRAMP uses `dired-listing-switches' for ftp-connection Rasmus
2014-11-26 13:35 ` Michael Albinus
2014-11-26 23:45   ` Rasmus [this message]
2014-11-29 10:42     ` Michael Albinus
2014-11-29 11:16       ` Rasmus
2014-11-29 12:21         ` Rasmus
2014-11-29 12:47           ` Rasmus
2014-11-30 11:10         ` Michael Albinus
2014-11-30 12:43           ` Rasmus
2014-12-09 13:56             ` Michael Albinus
2014-12-10 13:16               ` Rasmus
2014-12-10 14:15                 ` Andreas Schwab
2014-12-10 15:28                   ` Rasmus
2014-12-10 16:49                     ` Michael Albinus

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=87h9xlil0f.fsf@gmx.us \
    --to=rasmus@gmx.us \
    --cc=19192@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 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).