You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Paul Burba <pt...@gmail.com> on 2011/06/16 20:47:27 UTC
Re: svn commit: r1132561 - /subversion/trunk/subversion/tests/cmdline/copy_tests.py
On Mon, Jun 6, 2011 at 5:38 AM, <sb...@apache.org> wrote:
> Author: sbutler
> Date: Mon Jun 6 09:38:30 2011
> New Revision: 1132561
>
> URL: http://svn.apache.org/viewvc?rev=1132561&view=rev
> Log:
> Make copy_tests.py 102 XFAIL on OS X because APR doesn't support
> case-insensitivity on Unix.
>
> * subversion/tests/cmdline/copy_tests.py
> (case_only_rename): Set XFAIL.
>
> Modified:
> subversion/trunk/subversion/tests/cmdline/copy_tests.py
>
> Modified: subversion/trunk/subversion/tests/cmdline/copy_tests.py
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/copy_tests.py?rev=1132561&r1=1132560&r2=1132561&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/tests/cmdline/copy_tests.py (original)
> +++ subversion/trunk/subversion/tests/cmdline/copy_tests.py Mon Jun 6 09:38:30 2011
> @@ -5176,6 +5176,8 @@ def copy_base_of_deleted(sbox):
> # Regression test for issue #3702: Unable to perform case-only rename
> # on windows.
> @Issue(3702)
> +# APR's apr_filepath_merge() with APR_FILEPATH_TRUENAME is broken on OS X.
> +@XFail(svntest.main.is_os_darwin)
> def case_only_rename(sbox):
> """case-only rename"""
Hi Stephen,
Since issue #3702 is fixed and this test still fails on OSX could you
possibly and a new issue describing the problem and then associating
this test with that issue? I'd add one myself, but don't follow
exactly what the problem is.
No worries if you can't, I'm just trying to keep the number of failing
tests with no associated failing issue to a minimum (preferably zero
:-)
Paul
Re: svn commit: r1132561 - /subversion/trunk/subversion/tests/cmdline/copy_tests.py
Posted by Paul Burba <pt...@gmail.com>.
On Thu, Jun 16, 2011 at 2:47 PM, Paul Burba <pt...@gmail.com> wrote:
> On Mon, Jun 6, 2011 at 5:38 AM, <sb...@apache.org> wrote:
>> Author: sbutler
>> Date: Mon Jun 6 09:38:30 2011
>> New Revision: 1132561
>>
>> URL: http://svn.apache.org/viewvc?rev=1132561&view=rev
>> Log:
>> Make copy_tests.py 102 XFAIL on OS X because APR doesn't support
>> case-insensitivity on Unix.
>>
>> * subversion/tests/cmdline/copy_tests.py
>> (case_only_rename): Set XFAIL.
>>
>> Modified:
>> subversion/trunk/subversion/tests/cmdline/copy_tests.py
>>
>> Modified: subversion/trunk/subversion/tests/cmdline/copy_tests.py
>> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/copy_tests.py?rev=1132561&r1=1132560&r2=1132561&view=diff
>> ==============================================================================
>> --- subversion/trunk/subversion/tests/cmdline/copy_tests.py (original)
>> +++ subversion/trunk/subversion/tests/cmdline/copy_tests.py Mon Jun 6 09:38:30 2011
>> @@ -5176,6 +5176,8 @@ def copy_base_of_deleted(sbox):
>> # Regression test for issue #3702: Unable to perform case-only rename
>> # on windows.
>> @Issue(3702)
>> +# APR's apr_filepath_merge() with APR_FILEPATH_TRUENAME is broken on OS X.
>> +@XFail(svntest.main.is_os_darwin)
>> def case_only_rename(sbox):
>> """case-only rename"""
>
> Hi Stephen,
>
> Since issue #3702 is fixed and this test still fails on OSX could you
> possibly and a new issue describing the problem and then associating
> this test with that issue? I'd add one myself, but don't follow
> exactly what the problem is.
>
> No worries if you can't, I'm just trying to keep the number of failing
> tests with no associated failing issue to a minimum (preferably zero
> :-)
>
> Paul
Oh, I should add that this is the last thing standing between us and
completion of the 1.7 release task 'Test Review: Determine which XFail
and WIP tests should remain so, and which need to be fixed before
release.' Every other xfailing test in the suite has an associated
issue with a target milestone.
Re: svn commit: r1132561 - /subversion/trunk/subversion/tests/cmdline/copy_tests.py
Posted by Paul Burba <pt...@gmail.com>.
On Thu, Jun 16, 2011 at 2:47 PM, Paul Burba <pt...@gmail.com> wrote:
> On Mon, Jun 6, 2011 at 5:38 AM, <sb...@apache.org> wrote:
>> Author: sbutler
>> Date: Mon Jun 6 09:38:30 2011
>> New Revision: 1132561
>>
>> URL: http://svn.apache.org/viewvc?rev=1132561&view=rev
>> Log:
>> Make copy_tests.py 102 XFAIL on OS X because APR doesn't support
>> case-insensitivity on Unix.
>>
>> * subversion/tests/cmdline/copy_tests.py
>> (case_only_rename): Set XFAIL.
>>
>> Modified:
>> subversion/trunk/subversion/tests/cmdline/copy_tests.py
>>
>> Modified: subversion/trunk/subversion/tests/cmdline/copy_tests.py
>> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/copy_tests.py?rev=1132561&r1=1132560&r2=1132561&view=diff
>> ==============================================================================
>> --- subversion/trunk/subversion/tests/cmdline/copy_tests.py (original)
>> +++ subversion/trunk/subversion/tests/cmdline/copy_tests.py Mon Jun 6 09:38:30 2011
>> @@ -5176,6 +5176,8 @@ def copy_base_of_deleted(sbox):
>> # Regression test for issue #3702: Unable to perform case-only rename
>> # on windows.
>> @Issue(3702)
>> +# APR's apr_filepath_merge() with APR_FILEPATH_TRUENAME is broken on OS X.
>> +@XFail(svntest.main.is_os_darwin)
>> def case_only_rename(sbox):
>> """case-only rename"""
>
> Hi Stephen,
>
> Since issue #3702 is fixed and this test still fails on OSX could you
> possibly and a new issue describing the problem and then associating
> this test with that issue? I'd add one myself, but don't follow
> exactly what the problem is.
>
> No worries if you can't, I'm just trying to keep the number of failing
> tests with no associated failing issue to a minimum (preferably zero
> :-)
>
> Paul
Oh, I should add that this is the last thing standing between us and
completion of the 1.7 release task 'Test Review: Determine which XFail
and WIP tests should remain so, and which need to be fixed before
release.' Every other xfailing test in the suite has an associated
issue with a target milestone.