Re: The "follow copy history" initiative

Shlomi Fish wrote:

>On Wed, 31 Mar 2004, Ben Collins-Sussman wrote:
>
>
>
>>Step #3 is something that Shlomi Fish is interested in working on,
>>possibly on a branch. The idea is that we add a new RA function to the
>>bottom of our RA vtable (so as to be 1.1 compliant) which asks the
>>*repository* to do the history search using the fs_history API. That
>>way the client gets back a single targeted answer, rather than sloughing
>>through hundreds of logs. Once the new RA function exists, we upgrade
>>the new client function (created in step #1) to use it instead of
>>RA->get_logs().
>>
>>
>>
>
>Here is my initial patch to it. I implemented an svn_fs function to do
>just that. I also added a function to svn_ra_plugin_t to call it, and
>binded it only in ra_local (for the time being).
>
>

I have a problem with this patch: it only looks in the _past_. That
restriction is unnecessary (and evil :-), you should be able to peg
/foo/bar/baz@100 and ask where it is at revision 200.

I also don't like the name of the funciton, obviously, because it's not
just about files. I suggest svn_fs_get_node_path.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org