Re: tramp (2.1.19): Problem with tramp_vc_registered_read_file_names

[David Rothenberger]

On 10/19/2010 8:37 AM, Michael Albinus wrote:
> David Rothenberger writes:
> 
>>> Yes, tramp_vc_registered_read_file_names has been changed with Tramp
>>> 2.1.19, using here documents. Could you, please, set tramp-verbose to
>>> 10, and rerun the test? I'ld like to see the debug buffer then.
>>
>> I've attached the debug buffer and the comm buffer after the failure.
> 
> Thanks.
> 
> I've committed a patch to the Tramp repository, which might fix
> it. Could you, please, check?
> 
> Note, that the version in the repository is 2.2.0-pre, which means also
> a reorganization of Tramp's files. You need a new "make" step first.
> 
> Best regards, Michael.
 
The fix seems to work just fine.

I generally build a package for Cygwin that I then install using
it's installer. I had a few problems with my build script. I had to
apply a patch to fix a build issue with "tramp-version" not being
defined. The patch also builds and installs tramp-loaddefs.

Once I got the package and could load it, I did have a problem
opening files, since "tramp-handle-call-process-region" was
undefined. The second patch solved that problem for me.

----------------------------------------------------------------------
                              Patch 1
----------------------------------------------------------------------
Index: tramp/lisp/tramp.el
===================================================================
--- tramp.orig/lisp/tramp.el    2010-10-05 07:06:01.000000000 -0700
+++ tramp/lisp/tramp.el 2010-10-19 10:33:04.761125000 -0700
@@ -58,6 +58,7 @@
 
 ;;; Code:
 
+(require 'trampver)
 (require 'tramp-compat)
 
 ;;; User Customizable Internal Variables:
Index: tramp/lisp/Makefile.in
===================================================================
--- tramp.orig/lisp/Makefile.in 2010-10-19 10:33:04.370500000 -0700
+++ tramp/lisp/Makefile.in      2010-10-19 10:33:58.401750000 -0700
@@ -75,7 +75,8 @@
 endif
 
 INSTALL_FILES  = $(LISP_FILES) $(CONTRIB_FILES) \
-                 $(addsuffix .elc, $(basename $(LISP_FILES) $(CONTRIB_FILES)))
+                 $(addsuffix .elc, $(basename $(LISP_FILES) $(CONTRIB_FILES))) 
\
+                  tramp-loaddefs.el tramp-loaddefs.elc
 
 ifeq ($(EMACS_INFO), emacs)
   EM = $(EMACS) --no-site-file -batch
@@ -88,7 +89,7 @@
 
 .SUFFIXES: .elc .el
 
-all: autoloads $(addsuffix .elc, $(basename $(LISP_FILES) $(CONTRIB_FILES)))
+all: autoloads $(addsuffix .elc, $(basename $(LISP_FILES) $(CONTRIB_FILES))) 
tramp-loaddefs.elc
 
 autoloads: $(srcdir)/tramp-loaddefs.el
 $(srcdir)/tramp-loaddefs.el: $(LISP_FILES)

----------------------------------------------------------------------
                              Patch 2
----------------------------------------------------------------------
Index: tramp/lisp/tramp-util.el
===================================================================
--- tramp.orig/lisp/tramp-util.el       2010-09-15 13:14:11.000000000 -0700
+++ tramp/lisp/tramp-util.el    2010-10-19 10:47:01.805875000 -0700
@@ -128,7 +128,7 @@
              '(tramp-sh-file-name-handler
                tramp-fish-file-name-handler))
        (setq ad-return-value
-             (apply 'tramp-handle-call-process-region (ad-get-args 0)))
+             (apply 'call-process-region (ad-get-args 0)))
       ad-do-it))
   (add-hook 'tramp-util-unload-hook
            '(lambda () (ad-unadvise 'call-process-region)))

-- 
David Rothenberger  ----  address@hidden

"Been through Hell?  Whaddya bring back for me?"
                -- A. Brilliant