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

Re: svn commit: r33704 - trunk/subversion/libsvn_client

From: Paul Burba <ptburba_at_gmail.com>
Date: Mon, 20 Oct 2008 11:27:23 -0400

On Fri, Oct 17, 2008 at 2:37 PM, Greg Stein <gstein_at_gmail.com> wrote:
> On Thu, Oct 16, 2008 at 10:54 AM, <pburba_at_tigris.org> wrote:
>>...
>> * subversion/libsvn_client/merge.c
>> (filter_self_referential_mergeinfo): Don't try to filter self-referential
>> mergeinfo on paths that are scheduled for replacement.
>
> Another (recent) case where the log message has more information than
> what can be found in the code(!)
>
>>...
>> +++ trunk/subversion/libsvn_client/merge.c Thu Oct 16 10:54:33 2008 (r33704)
>> @@ -557,7 +557,8 @@ filter_self_referential_mergeinfo(apr_ar
>> /* If PATH itself is newly added there is no need to filter. */
>
> This comment is wrong, given the change below.

Fixed in r33780.

Paul

>> SVN_ERR(svn_wc__entry_versioned(&target_entry, path, adm_access,
>> FALSE, pool));
>> - if (target_entry->schedule == svn_wc_schedule_add)
>> + if (target_entry->schedule == svn_wc_schedule_add
>> + || target_entry->schedule == svn_wc_schedule_replace)
>> return SVN_NO_ERROR;
>>
>> adjusted_props = apr_array_make(pool, (*props)->nelts, sizeof(svn_prop_t));
>
>
> Cheers,
> -g

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-10-20 17:27:38 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.