Re: Dependency problem?

[Larry Jones]

Andrew Moise writes:
> 
> cvs_DEPENDENCIES = $(cvs_client_objects)
> 
>   (... uh-huh ...)

No, that's not correct.  It should be something like:

        cvs_DEPENDENCIES = ../diff/libdiff.a ../lib/libcvs.a $(ZLIB_LIBS)

This seems to work correctly on the cvs1-11-x-branch (which is what I
was looking at) but be broken on the trunk.  Removing the definition
from Makefile.am and letting automake generate it itself seems to do the
trick.

>   That's a sensible approach, yes.  The patch I sent is the wrong
> thing (TM), since it adds a bit of special-case nonsense to the diff
> code, but as I said I don't have a sufficient understanding of the code
> to remove the newline-addition hackery, and the current behavior is
> definitely problematic.
>   The problem _is_ specific to cvs; diff is not affected.

Then I misunderstood, I thought it was a generic diff bug that was
manifested in CVS.

-Larry Jones

What a stupid world. -- Calvin