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

RA->get_repos_root() needed

From: mark benedetto king <mbk_at_lowlatency.com>
Date: 2003-10-13 04:46:21 CEST

The svn blame implementation committed in rev 7389 is flawed: it does not
handle blame that spans path changes.

I started down the path of fixing this (essentially, walking backwards from
the last revision, tracking the path changes as they are encountered).

The problem I ran into is this:

The log message receiver gets passed a hash of "changed_paths". The
keys on this hash are absolute repository-paths (e.g. "/trunk/INSTALL").
In order to know what key to used to inspect this hash for changes, I need
to be able to split "http://svn.collab.net/repos/svn/trunk/INSTALL" into its
repository-URL and path components.

There does not appear to be a straightforward way of doing this. I suspect
that if there were, issue-960 would have been closed by now.

I think that ra_file and ra_dav can produce their repository-URL reasonably
easily, but that ra_svn will need a minor protocol change. I'm willing to
do the work once we reach consensus.

Anyway, in the meantime, I have a nasty hack that will work (I'll iterate
through the keys, taking the shortest one with the longest tail-match on
my URL).

--ben

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Oct 13 04:47:07 2003

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.