Re: PATCH thoughts...

Alex Rousskov wrote:
> > So please voice your opinions on this issue to help me off this
> > fence.  Should PATCH require support for one diff format, and if so,
> > what?
> 
> Is there a diff format that accommodates all of these:
> 	- text patching with fuzzy offsets (like Larry Wall's patch)
> 	- binary patching based on offsets
> 	- patching not based on offsets

Larry Wall's patch (when given unified or context diffs) can be used
without fuzzy offsets (just disable fuzzy matching in the patcher),
and it can be used on binary files (ugly an inefficient, because it's
still line based and depends crucially on LF bytes).  It doesn't use
byte offsets in either case.

> If yes, use it at a SHOULD level. If not, then it looks like it would
> be a mistake to pick a "winner" at this time.

I agree, there isn't a suitable "winner" at this time.

What does DAV + Delta-V propose at the moment?

The obvious format for PATCH is to pick the same format that can be
_fetched_, CVS-style, to update a workspace.  Is that in Delta-V?

Another capability to aim for is a patch format that is helpful for
resolving fuzzy matches in a 3-way merge in the same way as CVS-style
merging -- again, choose the same as Delta-V.

Finally, it would be nice to have VCDIFF-style compression integrated,
although compressing normal patches works ok so it's not that important.

-- Jamie

Received on Friday, 30 April 2004 05:51:00 UTC