You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Philip Martin <ph...@wandisco.com> on 2011/09/02 19:06:10 UTC

Re: svn commit: r1164581 - /subversion/branches/1.7.x/STATUS

cmpilato@apache.org writes:

> Author: cmpilato
> Date: Fri Sep  2 15:30:04 2011
> New Revision: 1164581
>
> URL: http://svn.apache.org/viewvc?rev=1164581&view=rev
> Log:
> Propose the issue #4003 fixes for backport.
>
> Modified:
>     subversion/branches/1.7.x/STATUS
>
> Modified: subversion/branches/1.7.x/STATUS
> URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x/STATUS?rev=1164581&r1=1164580&r2=1164581&view=diff
> ==============================================================================
> --- subversion/branches/1.7.x/STATUS (original)
> +++ subversion/branches/1.7.x/STATUS Fri Sep  2 15:30:04 2011
> @@ -128,6 +128,15 @@ Candidate changes:
>     Votes:
>       +1: rhuijben
>  
> + * r1164517, r1164580
> +   Fix issue #4003 ("'svn patch' seems unable to delete properties"). 
> +   Justification:
> +     This fixes a lack of consistency with how emptied file contents
> +     are handled.
> +   Notes:
> +     r1164517 adds the XFail regression test.
> +     r1164580 fixes the bug and expects test passage.
> +

Did you intend to vote for the above, or is it combined with the change
below?

>   * r1164535, r1164554
>     Fix issue #4004 ("'svn patch --reverse-diff' chokes on patches
>     which add files with props").
> @@ -135,7 +144,7 @@ Candidate changes:
>       Prevents failure from a use-case that folks will naturally expect
>       to work: "svn diff > PATCH && svn patch PATH --reverse-diff"
>     Notes:
> -     r1164535 adds the XFail regression test.
> +     r1164535 adds the XFail regression tests.
>       r1164554 fixes the bug and expects test passage.
>     Votes:
>       +1: cmpilato

-- 
Philip

Re: svn commit: r1164581 - /subversion/branches/1.7.x/STATUS

Posted by "C. Michael Pilato" <cm...@collab.net>.
On 09/02/2011 01:06 PM, Philip Martin wrote:

>> +   Fix issue #4003 ("'svn patch' seems unable to delete properties"). 
>> +   Justification:
>> +     This fixes a lack of consistency with how emptied file contents
>> +     are handled.
>> +   Notes:
>> +     r1164517 adds the XFail regression test.
>> +     r1164580 fixes the bug and expects test passage.
>> +
> 
> Did you intend

I just forgot to vote.  Thanks, Philip.

-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand


Re: svn commit: r1164581 - /subversion/branches/1.7.x/STATUS

Posted by "C. Michael Pilato" <cm...@collab.net>.
On 09/02/2011 01:06 PM, Philip Martin wrote:

>> +   Fix issue #4003 ("'svn patch' seems unable to delete properties"). 
>> +   Justification:
>> +     This fixes a lack of consistency with how emptied file contents
>> +     are handled.
>> +   Notes:
>> +     r1164517 adds the XFail regression test.
>> +     r1164580 fixes the bug and expects test passage.
>> +
> 
> Did you intend

I just forgot to vote.  Thanks, Philip.

-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand