From: Michael Albinus <michael.albinus@gmx.de>
Cc: eliz@gnu.org, romain@orebokech.com, monnier@iro.umontreal.ca,
emacs-devel@gnu.org
Subject: Re: Emacs unconditionally loads tramp.el
Date: Sat, 29 Apr 2006 21:55:39 +0200 [thread overview]
Message-ID: <87zmi4i2bo.fsf@gmx.de> (raw)
In-Reply-To: <E1FZup2-0000wZ-4W@fencepost.gnu.org> (Richard Stallman's message of "Sat, 29 Apr 2006 15:09:40 -0400")
Richard Stallman <rms@gnu.org> writes:
> I just found that this problem was due to XEmacs compatibility code.
> So I deleted that code. I won't delete XEmacs compatibility code
> merely because it is XEmacs compatibility code; but I won't let
> a bug remain in Emacs for the sake of XEmacs.
Could we agree on this?
magdalene:~/src/emacs/lisp/net> cvs diff tramp.el
Index: tramp.el
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/net/tramp.el,v
retrieving revision 1.95
diff -u -r1.95 tramp.el
--- tramp.el 29 Apr 2006 15:58:31 -0000 1.95
+++ tramp.el 29 Apr 2006 19:50:57 -0000
@@ -1931,9 +1931,14 @@
;; Handlers for partial tramp file names. For Emacs just
;; `file-name-all-completions' is needed.
(defconst tramp-completion-file-name-handler-alist
- '(
- (file-name-all-completions . tramp-completion-handle-file-name-all-completions)
- (file-name-completion . tramp-completion-handle-file-name-completion))
+ (append
+ '((file-name-all-completions . tramp-completion-handle-file-name-all-completions)
+ (file-name-completion . tramp-completion-handle-file-name-completion))
+ (when (featurep 'xemacs)
+ '((file-name-directory . tramp-completion-handle-file-name-directory)
+ (file-name-nondirectory . tramp-completion-handle-file-name-nondirectory)
+ (file-exists-p . tramp-completion-handle-file-exists-p)
+ (expand-file-name . tramp-completion-handle-expand-file-name))))
"Alist of completion handler functions.
Used for file names matching `tramp-file-name-regexp'. Operations not
mentioned here will be handled by `tramp-file-name-handler-alist' or the
> I also noticed this:
>
> ; These values conform to `file-attributes' from XEmacs 21.2.
> ; Emacs and other tools not checked.
> (defconst tramp-file-mode-type-map
>
> What exactly hasn't been checked here?
Don't know, it was written before I've entered the Tramp team. I'll
check.
Best regards, Michael.
next prev parent reply other threads:[~2006-04-29 19:55 UTC|newest]
Thread overview: 67+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-24 9:16 Emacs unconditionally loads tramp.el Romain Francoise
2006-04-24 11:33 ` Michael Albinus
2006-04-24 12:12 ` Romain Francoise
2006-04-24 13:18 ` Michael Albinus
2006-04-24 13:33 ` Stefan Monnier
2006-04-24 13:49 ` Romain Francoise
2006-04-24 19:43 ` Michael Albinus
2006-04-25 16:48 ` Richard Stallman
2006-04-25 18:10 ` Michael Albinus
2006-04-25 18:44 ` Stefan Monnier
2006-04-28 4:11 ` Michael Albinus
2006-04-26 3:56 ` Richard Stallman
2006-04-28 4:26 ` Michael Albinus
2006-04-28 6:15 ` Eli Zaretskii
2006-04-28 20:01 ` Michael Albinus
2006-04-29 14:26 ` Eli Zaretskii
2006-04-29 16:09 ` Michael Albinus
2006-04-29 16:55 ` Eli Zaretskii
2006-04-29 18:30 ` Michael Albinus
2006-04-29 18:57 ` Michael Price
2006-04-29 19:10 ` Michael Albinus
2006-04-29 20:47 ` Miles Bader
2006-04-29 21:37 ` Michael Albinus
2006-04-29 22:50 ` Miles Bader
2006-04-30 8:23 ` Michael Albinus
2006-04-29 21:40 ` David Kastrup
2006-04-29 22:12 ` Drew Adams
2006-04-29 22:49 ` Jason Rumney
2006-04-29 22:58 ` Lennart Borgman
2006-04-30 11:18 ` Oscar Fuentes
2006-04-30 12:13 ` Jason Rumney
2006-04-30 13:03 ` Oscar Fuentes
2006-04-30 17:46 ` Eli Zaretskii
2006-04-29 23:33 ` Drew Adams
2006-04-30 3:45 ` Eli Zaretskii
2006-04-30 14:18 ` Drew Adams
2006-04-30 14:47 ` Michael Price
2006-04-30 16:31 ` Eli Zaretskii
2006-04-30 16:51 ` Drew Adams
2006-04-30 17:36 ` Eli Zaretskii
2006-04-30 17:41 ` Drew Adams
2006-04-30 18:06 ` Michael Albinus
2006-04-30 18:08 ` Drew Adams
2006-04-30 5:26 ` Stefan Monnier
2006-04-30 14:11 ` Drew Adams
2006-04-30 6:50 ` Tramp & Cygwin (was: Emacs unconditionally loads tramp.el) Michael Albinus
2006-04-30 12:12 ` Tramp & Cygwin Jason Rumney
2006-04-30 12:28 ` Michael Albinus
2006-04-30 14:10 ` Drew Adams
2006-04-30 14:19 ` Tramp & Cygwin (was: Emacs unconditionally loads tramp.el) Drew Adams
2006-04-30 16:11 ` Eli Zaretskii
2006-04-30 16:45 ` Tramp & Cygwin Michael Albinus
2006-04-30 17:19 ` Eli Zaretskii
2006-04-30 13:04 ` Tramp performance (was: Emacs unconditionally loads tramp.el) Michael Albinus
2006-04-30 3:47 ` Emacs unconditionally loads tramp.el Eli Zaretskii
2006-04-30 7:00 ` Michael Albinus
2006-04-30 16:16 ` Eli Zaretskii
2006-04-28 15:45 ` Richard Stallman
2006-04-28 19:58 ` Michael Albinus
2006-04-29 14:31 ` Eli Zaretskii
2006-05-01 13:45 ` Michael Albinus
2006-04-29 19:09 ` Richard Stallman
2006-04-29 19:55 ` Michael Albinus [this message]
2006-04-30 3:04 ` Richard Stallman
2006-05-01 13:28 ` Michael Albinus
2006-04-24 17:52 ` Richard Stallman
2006-04-24 19:48 ` Michael Albinus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87zmi4i2bo.fsf@gmx.de \
--to=michael.albinus@gmx.de \
--cc=eliz@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=monnier@iro.umontreal.ca \
--cc=romain@orebokech.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).