Re: Quotes in Dired listing switches

[Michael Albinus]

Stefan Monnier <address@hidden> writes:

> Obviously, there's a problem in the inconsistent parsing of
> dired-listing-switches, where the basic code splits it at spaces without
> doing any additional unquoting or analysis, whereas the Tramp code seems
> to just pass it as-is to the shell (so if you set it to "-al $(rm -rf ~/.)"
> it will try to do something funny).
> I won't claim that the basic code's naive splitting is great, but
> Tramp's similarly naive use is not great either.  So, I suggest to treat
> it as a bug in Tramp which should split it like the basic code and then
> shell-quote-argument the parts.

In tramp.el, I've added quoting of the "'" character in SWITCHES, passed
by `insert-directory', which shall be sufficient.

However, remote directory listings won't show a "thousands separator of
the current locale", because Tramps sets LC_ALL to "C" on the remote
hosts. The C locale has no thousands separator. If you (Juri) want to
change this, you might customize `tramp-remote-process-environment',
setting LC_ALL to en_US.UTF-8 or so. But please be careful, because
Tramp reads output from some commands, and might expect English.

>         Stefan

Best regards, Michael.