bug#5840: 24.0.50; wrong tramp-file-name-regexp

[Michael Albinus <michael.albinus@gmx.de>]

Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:

> Hi,

Hi,

> with these settings:
>
> ,----
> | (add-hook 'minibuffer-setup-hook
> |           (lambda () (add-hook 'post-command-hook
> |                                'minibuffer-completion-help nil t)))
> `----
>
> If you do:
> C-x C-f ==> /su:
>
> You have not the time to type the second ":" and tramp fail.
> (same for ssh/ftp)
>
> This could be fixed by modifying tramp-file-name-regexp like this:
>
> (tramp-file-name-regexp "\\`/\\([^[/:]+\\|[^/]+]\\):.*:")

This would be wrong. "/hostname:/path" is a valid remote file name, which
would not be catched by your regexp.

(For the records, I also dislike Tramp's syntactic ambiguity. But this is
history, I guess we cannot change it anymore.)

In order to fix this problem, one could tweak `minibuffer-completion-predicate'
in `minibuffer-completion-help', which is `file-exists-p' in the given
case.

Best regards, Michael.