On Fri, May 23, 2008 at 11:39 AM, Mark Phippard <markphip_at_gmail.com> wrote:
> On Fri, May 23, 2008 at 11:33 AM, Bert Huijben <bert_at_vmoo.com> wrote:
>>> > But I don't know about a need of notification API
>>> > incompatibility between trunk and 1.5.x.
>>
>> This test probably fails on the notification changes in r31211. Before that
>> revision the notify handler received commit paths which weren't valid in all
>> cases (not-relative and not-absolute and it must be one or the other to be
>> correct).
>>
>> Starting in r31211 the handler receives the whole path and a path_prefix of
>> the commit root. (From those parts the old displayed path can be
>> calculated.. but that display path is not always valid)
>>
>> See subversion/svn/notify.c for some example code on how to translate the
>> path.
>>
>> (Summary: If path prefix is non-null and equals the start of path, then
>> strip prefix from path).
>>
>>
>> Note: r31211 is not backported to 1.5.x.
>
> So this doesn't really explain why the test would fail on the branch.
> Did a test that only belongs on trunk get backported to the branch?
It looks like this test was added to Ruby in r31348 (after your
change) and backported to 1.5.x in r31356. So I propose that we just
commit the 1-line patch that Kouhei submitted directly to the 1.5.x
branch as the resolution for this.
--
Thanks
Mark Phippard
http://markphip.blogspot.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-23 21:35:16 CEST