You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ph...@apache.org on 2011/04/05 02:01:04 UTC

svn commit: r1088828 - /subversion/trunk/subversion/tests/cmdline/revert_tests.py

Author: philip
Date: Tue Apr  5 00:01:03 2011
New Revision: 1088828

URL: http://svn.apache.org/viewvc?rev=1088828&view=rev
Log:
* subversion/tests/cmdline/revert_tests.py
  (revert_permissions_only): Skip on non-POSIX platforms.

Modified:
    subversion/trunk/subversion/tests/cmdline/revert_tests.py

Modified: subversion/trunk/subversion/tests/cmdline/revert_tests.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/revert_tests.py?rev=1088828&r1=1088827&r2=1088828&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/revert_tests.py (original)
+++ subversion/trunk/subversion/tests/cmdline/revert_tests.py Tue Apr  5 00:01:03 2011
@@ -1086,6 +1086,8 @@ def revert_non_recusive_after_delete(sbo
   expected_status.tweak('A/B/E', status='  ')
   svntest.actions.run_and_verify_status(wc_dir, expected_status)
 
+# I suspect svn_io__is_finfo_read_only doesn't work on Windows :-(
+@SkipUnless(svntest.main.is_posix_os)
 def revert_permissions_only(sbox):
   "permission-only reverts"
 



RE: svn commit: r1088828 - /subversion/trunk/subversion/tests/cmdline/revert_tests.py

Posted by Bert Huijben <be...@qqmail.nl>.

> -----Original Message-----
> From: philip@apache.org [mailto:philip@apache.org]
> Sent: dinsdag 5 april 2011 2:01
> To: commits@subversion.apache.org
> Subject: svn commit: r1088828 -
> /subversion/trunk/subversion/tests/cmdline/revert_tests.py
> 
> Author: philip
> Date: Tue Apr  5 00:01:03 2011
> New Revision: 1088828
> 
> URL: http://svn.apache.org/viewvc?rev=1088828&view=rev
> Log:
> * subversion/tests/cmdline/revert_tests.py
>   (revert_permissions_only): Skip on non-POSIX platforms.
> 
> Modified:
>     subversion/trunk/subversion/tests/cmdline/revert_tests.py
> 
> Modified: subversion/trunk/subversion/tests/cmdline/revert_tests.py
> URL:
> http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/r
> evert_tests.py?rev=1088828&r1=1088827&r2=1088828&view=diff
> ==========================================================
> ====================
> --- subversion/trunk/subversion/tests/cmdline/revert_tests.py (original)
> +++ subversion/trunk/subversion/tests/cmdline/revert_tests.py Tue Apr  5
> 00:01:03 2011
> @@ -1086,6 +1086,8 @@ def revert_non_recusive_after_delete(sbo
>    expected_status.tweak('A/B/E', status='  ')
>    svntest.actions.run_and_verify_status(wc_dir, expected_status)
> 
> +# I suspect svn_io__is_finfo_read_only doesn't work on Windows :-(
> +@SkipUnless(svntest.main.is_posix_os)
>  def revert_permissions_only(sbox):
>    "permission-only reverts"

Finfo readonly does work, but you can't set a file to readonly in python with the same chmod as on posix as Windows uses access control lists instead of [user, group, other].

I'll look into this tomorrow.

	Bert