* bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
@ 2010-09-16 6:32 Thierry Volpiatto
2010-09-16 8:35 ` Michael Albinus
2010-09-16 17:06 ` Thierry Volpiatto
0 siblings, 2 replies; 8+ messages in thread
From: Thierry Volpiatto @ 2010-09-16 6:32 UTC (permalink / raw)
To: 7040
Hi,
tramp-handle-directory-files is not loaded.
To reproduce, try from remote to copy recursively a directory.
--
A+ Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
2010-09-16 6:32 bug#7040: 24.0.50; tramp-handle-directory-files is not loaded Thierry Volpiatto
@ 2010-09-16 8:35 ` Michael Albinus
2010-09-16 9:59 ` Thierry Volpiatto
2010-09-16 17:06 ` Thierry Volpiatto
1 sibling, 1 reply; 8+ messages in thread
From: Michael Albinus @ 2010-09-16 8:35 UTC (permalink / raw)
To: Thierry Volpiatto; +Cc: 7040
Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:
> Hi,
Hi,
> tramp-handle-directory-files is not loaded.
>
> To reproduce, try from remote to copy recursively a directory.
Works for me. Maybe, tramp-loaddefs.el is not regenerated? Try
# cd lisp; make autoloads
If this doesn't help, I need at least to know, which remote method you
have used. ssh? scp?
Best regards, Michael.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
2010-09-16 8:35 ` Michael Albinus
@ 2010-09-16 9:59 ` Thierry Volpiatto
2010-09-16 10:14 ` Michael Albinus
0 siblings, 1 reply; 8+ messages in thread
From: Thierry Volpiatto @ 2010-09-16 9:59 UTC (permalink / raw)
To: Michael Albinus; +Cc: 7040
Hi Michael,
Michael Albinus <michael.albinus@gmx.de> writes:
> Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:
>
>> Hi,
>
> Hi,
>
>> tramp-handle-directory-files is not loaded.
>>
>> To reproduce, try from remote to copy recursively a directory.
>
> Works for me. Maybe, tramp-loaddefs.el is not regenerated?
Don't know, is the makefile aware of this file?
> 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)
> If this doesn't help, I need at least to know, which remote method you
> have used. ssh? scp?
su, but i guess it's the same with ssh methods.
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.
> Best regards, Michael.
>
--
A+ Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
2010-09-16 9:59 ` Thierry Volpiatto
@ 2010-09-16 10:14 ` Michael Albinus
0 siblings, 0 replies; 8+ messages in thread
From: Michael Albinus @ 2010-09-16 10:14 UTC (permalink / raw)
To: Thierry Volpiatto; +Cc: 7040@debbugs.gnu.org
Thierry Volpiatto <thierry.volpiatto@gmail.com> 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?
> 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). 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 ...)
This shall also work with Emacs 23.
Best regards, Michael.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
2010-09-16 6:32 bug#7040: 24.0.50; tramp-handle-directory-files is not loaded Thierry Volpiatto
2010-09-16 8:35 ` Michael Albinus
@ 2010-09-16 17:06 ` Thierry Volpiatto
2010-09-16 21:08 ` Michael Albinus
1 sibling, 1 reply; 8+ messages in thread
From: Thierry Volpiatto @ 2010-09-16 17:06 UTC (permalink / raw)
To: bug-gnu-emacs
Michael Albinus <michael.albinus@gmx.de> writes:
> Thierry Volpiatto <thierry.volpiatto@gmail.com> 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
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
2010-09-16 17:06 ` Thierry Volpiatto
@ 2010-09-16 21:08 ` Michael Albinus
2010-09-16 21:19 ` Thierry Volpiatto
0 siblings, 1 reply; 8+ messages in thread
From: Michael Albinus @ 2010-09-16 21:08 UTC (permalink / raw)
To: Thierry Volpiatto; +Cc: bug-gnu-emacs
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.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
2010-09-16 21:08 ` Michael Albinus
@ 2010-09-16 21:19 ` Thierry Volpiatto
2010-09-17 20:53 ` Michael Albinus
0 siblings, 1 reply; 8+ messages in thread
From: Thierry Volpiatto @ 2010-09-16 21:19 UTC (permalink / raw)
To: Michael Albinus; +Cc: bug-gnu-emacs
Michael Albinus <michael.albinus@gmx.de> writes:
> 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).
Well, i don't like defadvice, so i will see if i maintain this as
emacs23.2 is a stable emacs version.
Thank you anyway for your help and advices.
>> 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?
Yes.
> Best regards, Michael.
>
--
A+ Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#7040: 24.0.50; tramp-handle-directory-files is not loaded
2010-09-16 21:19 ` Thierry Volpiatto
@ 2010-09-17 20:53 ` Michael Albinus
0 siblings, 0 replies; 8+ messages in thread
From: Michael Albinus @ 2010-09-17 20:53 UTC (permalink / raw)
To: Thierry Volpiatto; +Cc: 7040-done
Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:
>> So we can close the bug report?
>
> Yes.
Done.
Best regards, Michael.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2010-09-17 20:53 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-16 6:32 bug#7040: 24.0.50; tramp-handle-directory-files is not loaded Thierry Volpiatto
2010-09-16 8:35 ` Michael Albinus
2010-09-16 9:59 ` Thierry Volpiatto
2010-09-16 10:14 ` Michael Albinus
2010-09-16 17:06 ` Thierry Volpiatto
2010-09-16 21:08 ` Michael Albinus
2010-09-16 21:19 ` Thierry Volpiatto
2010-09-17 20:53 ` Michael Albinus
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).