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: Thierry Volpiatto
Subject: bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
Date: Thu, 16 Sep 2010 19:06:18 +0200
User-agent: Gnus/5.110011 (No Gnus v0.11) Emacs/24.0.50 (gnu/linux)

Michael Albinus <address@hidden> writes:

> Thierry Volpiatto <address@hidden> writes:
>
>> Hi Michael,
>
> Hi Thierry,
>
>>> Works for me. Maybe, tramp-loaddefs.el is not regenerated? 
>> Don't know, is the makefile aware of this file? 
>
> It should, I have committed the patch at once, including Makefile.in
> changes. However, a simple "make" does NOT generate autoloads (not only
> for Tramp).
>
>>> Try # cd lisp; make autoloads
>> Sorry, i can't do that yet as the emacs24 i have is installed by gentoo
>> from bzr, but the makefiles are removed in directory.
>> Maybe i can try to load the file itself? (need to switch i am on 23 now)
>
> Do you use Tramp outside the Emacs distribution? Have you downloaded
> from the Tramp repo?
No i use tramp that come with Emacs.

>> It's just a problem of loading the definition of
>> tramp-handle-directory-files, what i do here in anything-config.el
>> because i need this definition for compatibility with emacs versions < 23.2.
>> So it is working fine with this definition loaded.
>
> 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.

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 ;-)


-- 
A+ Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997 






reply via email to

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