On Sun, Aug 15, 2010 at 09:27:22PM +0200, Daniel Näslund wrote:
> Hi!
> 
> Why doesn't svn_ra_do_diff3() request that the repository sends copyfrom
> args? The best explanation I've found is this commit message:
> 
> ------------------------------------------------------------------------
> r866577 | sussman | 2007-09-10 06:56:55 +0200 (mån, 10 sep 2007) | 87 lines
> 
> Send copyfrom-args on updates *only* if the client explicitly requests them.
> 
> This keeps the 1.5 servers backward-compatible.  Because incoming
> copyfrom args cause old clients to error, they should only be
> requested by new clients that understand them.  This means adding a
> new boolean 'send_copyfrom_args' to the main libsvn_repos
> editor-driver, svn_ra_do_update2(), and all four RA implementations.
> Eesh!
> 
> [.. Rest of log message omitted for brevity ..]
> 
> I'm assuming noone saw a use case for having diff display copies back then and
> since it involved passing down a lot of parameters, only svn_ra_do_updateX()
> was revved. But with git diffs we have a use case for copyfrom info. How about
> enable it?

Yes, let's enable it. In the same backwards-compatible fashion of course.

Stefan

Reply via email to