On Fri, Mar 20, 2009 at 00:15, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
> Daniel Shahaf wrote on Thu, 19 Mar 2009 at 23:25 +0200:
>> CGreg Stein wrote on Thu, 19 Mar 2009 at 21:20 +0100:
>> > I don't know of a good way to test this code... not familiar with this
>> > crop code. If somebody can set up a test that fails before / succeeds
>> > after this change, then that would be great. I'd be happy to help with
>> > this, but don't know where to start to get this functionality invoked.
>> >
>>
>> How about this?
>>
>> % cd trunk/A
>> % svn up --set-depth exclude C
>> % grep '^C$' .svn/entries
>> C
>> % svn up --set-depth exclude D
>> % grep '^C$' .svn/entries
>> % grep '^D$' .svn/entries
>> D
>> %
>>
>> (i.e., ./C is lost from the entries file, but ./D is)
>>
>> I didn't test if your change fixes it.
>>
>
> It does. r36689.
You Rock!! Thanks for the test!
That is excellent, and it demonstrates a real bug that a user may run
into. I'm gonna propose this as a backport for 1.6.1.
Thx for helping,
-g
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1359412
Received on 2009-03-20 01:35:31 CET