bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Michael Albinus
Subject: bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
Date: Thu, 16 Sep 2010 23:08:33 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux)

Thierry Volpiatto <address@hidden> 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.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]