Re: keep_local in harvest_committables

Neels J Hofmeyr <neels_at_elego.de> writes:

> On 2010-05-13 00:25, Philip Martin wrote:
>> Neels J Hofmeyr <neels_at_elego.de> writes:
>>
>>> The bottom line is: simply remove that madly misguided entry->keep_local
>>> conditional and always call bail_on_tree_conflicted_children(). The rest of
>>> the patch is mending the test suite to be sensible about it. Will do.
>>
>> See r943722 for a slightly different approach. If you want to change
>> it further that's fine by me.
>
> * neels feels bad for causing work by not reading dev@ for a few days

Don't worry, the time consuming bit was the url/entry stuff not
keep_local.

> Should we keep that temp func for the time being?

I don't mind. The whole 'svn rm --keep-local' with tree conflicts is
a bit suspect. After a commit the unversioned directory still
contains a .svn directory. Reverting instead of committing leads to
unversioned directories that should be versioned. See my other
thread.

-- 
Philip