bug#7040: 24.0.50; tramp-handle-directory-files is not loaded

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

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

>> Oops, it is not recommended to call `tramp-handle-*' functions directly
>> (you would bypass locking mechanisms).
>
> Yes, but i don't call this function directly, i just inline the actual
> tramp-handle-directory-files to allow users of old tramp/emacs version
> to use this version of the function that fix ".", "..", and sorting
> (what we have fixed some months ago).
>
> At first it was inlined inside a function with a flet, but i realized
> that it is not called when i copy/rename, etc.. files, so it is now
> (re)defined as defun.
>
> I think the definition inlined in flet was confusing tramp.

Maybe the best solution is defadvice. Then you don't need to redefine
Tramp's internals (and you are independent from further changes in that
function).

> I have now tried in emacs -Q (so without anything) and copy recursively
> from remote work fine.
>
>> With recent Tramp 2.2, those
>> functions have been moved to another package. Likely, that's why you
>> don't see them. Instead of, call
>>
>> (tramp-file-name-handler 'directory-files ...)
>
> Ok thanks, that's good to know, but as said above i don't use
> tramp-handle-directory-files directly, i let tramp do the job.
>
> Sorry for the noise, but things are clearer now, thanks ;-)

So we can close the bug report?

Best regards, Michael.