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

Re: 1.6.10 up for testing/signing

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Tue, 06 Apr 2010 21:14:05 -0400

C. Michael Pilato wrote:
> 1. All the RA layers depend on a helper function
> svn_mergeinfo__remove_prefix_from_catalog() to convert repos abspaths
> into paths relative to the session baton. Unfortunately, that function
> is a touch braindead and doesn't return consistent relpaths at all.

This is fixed on trunk.

Also on trunk is some code in the RA loader which can correct the broken
output from servers that haven't yet been fixed. This bit is proposed for
backport to 1.6.x.

> 2. libsvn_client/mergeinfo.c:get_mergeinfo(), which is used by several
> public APIs, claims to return a catalog keyed on repository relpaths.
> But in the is_url case, it ... uh ... doesn't.

This is fixed on trunk, and isn't relevant to the 1.6.x codebase.

Just to restate the original bug that led us here:

3. libsvn_client/mergeinfo.c:get_mergeinfo() passes
    (RA-session-rooted-at-some-relpath, relpath) to an underlying API
    where it should be passing (RA-session-rooted-at-some-relpath, "").

This is fixed on trunk, and proposed for backport to 1.6.x.

Currently, the 1.6.x-3242-partial-fixes branch passes all four RA layers and
JavaHL tests for me.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2010-04-07 03:14:45 CEST

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.