Stephen Leake <[EMAIL PROTECTED]> writes:

> My proposal is that dvc-diff does _not_ call dvc-apply, it just calls
> dvc-delta. So the back-ends only need to implement dvc-delta.

I started working on this, but gave up on bzr.

The differences between bzr-diff and bzr-delta are more in the user
interface than the back-end. None of the other back-ends use
'revision-diff, but perhaps they should. There are also other
differences among the back-ends in how they do the user interface.

So it makes more sense to pull the user interface stuff into the
front-end, rather than to try to merge these now. Then it will be
clearer what to do for the back-ends.

I did clean up dvc-diff; it no longer takes a base-rev, and the
docstring defines the args better, and refers to dvc-delta for
non-local diffs.

I fixed all calls to dvc-diff, and all back-end implementations to
match.

For some back-ends, that had implemented base-rev for dvc-diff, I
factored out a back-end-diff-1, and called that from backend-diff and
backend-delta. 

I tested this with mtn and bzr; I don't have the other back-ends
installed. 

-- 
-- Stephe

_______________________________________________
Dvc-dev mailing list
[email protected]
https://mail.gna.org/listinfo/dvc-dev

Reply via email to