[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Feature proposal: maintain timestamps on svn hotcopy

From: Scott Parkerson <scott.parkerson_at_gmail.com>
Date: 2005-03-24 05:37:42 CET

On Wed, 23 Mar 2005 23:00:27 +0000, Philip Martin
<philip@codematters.co.uk> wrote:

> You can't add a parameter to svn_io_copy_file as that would break the
> ABI. You need to rename the function, to svn_io_copy_file2 say, add
> the new parameter and implementation, and then write a wrapper with
> the original name and parameter list that calls the new function.

Oh, crud. I spent the afternoon adding the parameter to svn_io_copy_file
and trickling it upwards to all the callers -- a good twelve or so files were
changed. Good thing I asked. :)

Anyways, I'm still going to have to modify other stuff in io.c... what can
and can't I modify without breaking the ABI? Is there a rule of thumb I
can follow to make sure I don't screw things up for everyone else?
(My apologies for being a total n00b on this.)

> I don't think there is a specific test for svn_io_copy_file but the
> regression tests as whole exercise it. The hotcopy command gets run
> by subversion/tests/clients/cmdline/svnadmin_tests.py, you might like
> to add to that.



scott parkerson. geek, erstwhile prophet and fool.
just a cog in the machinery: http://www.smerpology.org/sprocket/
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Mar 24 05:38:52 2005

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.