Neels J Hofmeyr <ne...@elego.de> writes:

> On 2010-05-13 00:25, Philip Martin wrote:
>> Neels J Hofmeyr <ne...@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

Reply via email to