C-x v l on CVS file leads "Marker does not point anywhere" in experimental vc*.el

[Adrian Aichner]

Ville Skyttä <scop at xemacs.org> writes:

> On Mon, 2006-05-01 at 11:42 +0200, Adrian Aichner wrote:
>> Ville Skyttä <scop at xemacs.org> writes:
>>
>> > Do you still see this error with current CVS (the version of vc.el
>> > containing the missing log-view workaround)?
>> 
>> Hi Ville, I can't build your vc package in the package hierarchy.
>
> Sure you can ;)
>
>> Here is where it breaks:
>
> But that breakage concerns only the part where "make binkit" tries to
> modify package-index.  The tarball is created with "make binkit" and all
> files I care about are installed with "make install" anyway, so I've
> just happily ignored that error during the few times I've actually
> created a binkit out of the package (usually I just "make install" it).

Ah, but I wanted to verify the whole supply-line :-)

>
> Anyway, this is probably easy enough to fix if people think it's worth
> fixing for the unsupported area; for example the attached patch seems to
> do the trick.  What do you think?

Yup, I already committed a similar, but more general fix, which works
fine.

Some light testing shows your latest vc CVS sources work well for me.

In particular it fixes
C-x v l runs the command vc-print-log
and
M-x erevision
for me.

Looking forward to this major step for the vc package!

Adrian

>
>
>
> Index: XEmacs.rules
> ===================================================================
> RCS file: /pack/xemacscvs/XEmacs/packages/XEmacs.rules,v
> retrieving revision 1.56
> diff -u -r1.56 XEmacs.rules
> --- XEmacs.rules	3 Feb 2005 20:09:06 -0000	1.56
> +++ XEmacs.rules	1 May 2006 10:18:08 -0000
> @@ -519,7 +519,8 @@
>  		'$(AUTHOR_VERSION)' '$(MAINTAINER)' '$(CATEGORY)'
>          #  #### See if we can't CLEAN this up
>  	$(XEMACS_BATCH_DIRTY) \
> -		-l ../../hack-package-index.el -f batch-hack-package-index \
> +		-l $(XEMACS_PACKAGES_BASE)/hack-package-index.el \
> +			-f batch-hack-package-index \
>  			$(PACKAGE) package-info $(PACKAGE_INFO) $(CATEGORY)
>  
>  ifeq ($(BUILD_TARS),t)

-- 
Adrian Aichner
 mailto:adrian at xemacs.org
 http://www.xemacs.org/