10:19 p.m.
Hello Ville, this problem is still around.
Let me know if you need more information to fix this.
Best regards!
Adrian
Debugger entered--Lisp error: (void-function process-file)
process-file("whoami" nil t)
vc-user-login-name("/[sshx/adrian(a)www.xemacs.org]/etc/named/xemacs/db.xemacs")
vc-rcs-fetch-master-state("/[sshx/adrian(a)www.xemacs.org]/etc/named/xemacs/db.xemacs"
"1.74")
vc-rcs-state("/[sshx/adrian(a)www.xemacs.org]/etc/named/xemacs/db.xemacs")
apply(vc-rcs-state
"/[sshx/adrian(a)www.xemacs.org]/etc/named/xemacs/db.xemacs")
vc-call-backend(RCS state
"/[sshx/adrian(a)www.xemacs.org]/etc/named/xemacs/db.xemacs")
vc-recompute-state("/[sshx/adrian(a)www.xemacs.org]/etc/named/xemacs/db.xemacs")
vc-next-action-on-file("/[sshx/adrian(a)www.xemacs.org]/etc/named/xemacs/db.xemacs"
nil)
byte-code("..." [verbose vc-dired-mode buffer-file-name vc-parent-buffer files
dired-get-marked-files make-local-variable vc-dired-window-configuration
current-window-configuration mapconcat #<compiled-function (f) "...(14)" [f
file vc-state up-to-date "＠" ""] 2> ""
vc-next-action-dired nil "dummy" vc-start-entry "Enter a change comment for
the marked files." throw nogo pop-to-buffer vc-next-action-on-file error "Buffer
%s is not associated with a file" buffer-name] 7)
vc-next-action(nil)
call-interactively(vc-next-action)
--
Adrian Aichner
mailto:adrian＠xemacs.org
http://www.xemacs.org/
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-beta
8:55 p.m.
On Thursday 27 September 2007, Adrian Aichner wrote:
Hello Ville, this problem is still around.
Let me know if you need more information to fix this.
Best regards!
Adrian
Debugger entered--Lisp error: (void-function process-file)
process-file("whoami" nil t)
Bummer. Actually this is (to me, at least) kind of a new problem - turns out
my previous change in tramp which "fixed" this was somewhat naive and now we
run into a similar one in vc-hooks.
But in the absence of process-file or similar that we could use the tramp
handlers for, let's leave tramp as is (patched) and go all the way then in
vc - the attached patch is not that pretty but I think it should fix this
issue. Could you test it?
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-beta
9:58 p.m.
Ville Skyttä <scop(a)xemacs.org> writes:
But in the absence of process-file or similar that we could use the
tramp
handlers for, let's leave tramp as is (patched) and go all the way then in
vc - the attached patch is not that pretty but I think it should fix this
issue. Could you test it?
Sorry to enter your discussion. But I believe it is not a good idea to
have tramp-related code in vc*.el. In order to hide such dependencies,
tramp-vc.el was introduced.
I will do my best, but I cannot guarantee that the API will be kept
stable for internal Tramp functions. Therefore any code using Tramp
internal functions might fail in the future. And indeed,
`tramp-send-command-and-check' has a different parameter list in Tramp
2.1. And since Tramp 2.1 is also intended for XEmacs, your patch would
fail there already.
However, even if you take Tramp functionality directly, you should
minimize dependencies.
(defun vc-user-login-name (file)
"Return the name under which the user accesses the given FILE."
(or (and (eq (string-match tramp-file-name-regexp file) 0)
Hmm. In GNU Emacs (please forgive me) the function `file-remote-p'
supports file name handlers for magic file names. Such a call is much
more robust wrt Tramp internals.
(or (and (file-remote-p file)
I believe it might be a good idea for XEmacs to expand its own
`file-remote-p' such a way. But it isn't applicable just now, I
understand.
;; tramp case: execute "whoami" via tramp
- (let ((default-directory (file-name-directory file)))
- (with-temp-buffer
- (if (not (zerop (process-file "whoami" nil t)))
Maybe you can save this approach. All file name handler based functions
in Tramp are implemented as `tramp-handle-<function>' with exactly the
same parameter list. So you shall define something like this:
(or (functionp 'process-file)
(defalias 'process-file 'tramp-handle-process-file))
The other altrnative would be to call `tramp-handle-process-file' directly.
But this would be more work to remove this hack once XEmacs knows
`process-file' as well (given, you have other places to use this
function too).
- ;; fall through if "whoami" didn't
work
- nil
- ;; remove trailing newline
- (delete-region (1- (point-max)) (point-max))
- (buffer-string)))))
+ ;; XEmacs change: we don't have process-file or call-process for
+ ;; which we could apply the tramp handler, use tramp directly
+ (require 'tramp)
+ (with-parsed-tramp-file-name file nil
+ (if (not (zerop (tramp-send-command-and-check
+ multi-method method user host "whoami")))
As I said already, this won't work with Tramp 2.1 (it doesn't know
multi-method anymore).
Best regards, Michael.
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-beta
10:20 p.m.
Ville Skyttä <scop(a)xemacs.org> writes:
On Thursday 27 September 2007, Adrian Aichner wrote:
> Hello Ville, this problem is still around.
> Let me know if you need more information to fix this.
>
> Best regards!
>
> Adrian
>
> Debugger entered--Lisp error: (void-function process-file)
> process-file("whoami" nil t)
Bummer. Actually this is (to me, at least) kind of a new problem - turns out
my previous change in tramp which "fixed" this was somewhat naive and now we
run into a similar one in vc-hooks.
But in the absence of process-file or similar that we could use the tramp
handlers for, let's leave tramp as is (patched) and go all the way then in
vc - the attached patch is not that pretty but I think it should fix this
issue. Could you test it?
So done.
Thanks for the quick fix, Ville!
I tested under that same conditions which uncovered the problem, so
I'm convinced your fix works.
Best regards!
Adrian
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-beta
--
Adrian Aichner
mailto:adrian＠xemacs.org
http://www.xemacs.org/
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-beta
6055
days inactive
6056
days old
3 comments
3 participants
participants (3)
-
Adrian Aichner -
Michael Albinus -
Ville Skyttä