* 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).