Author: philip
Date: Wed Mar 16 09:49:27 2011
New Revision: 1082106

URL: http://svn.apache.org/viewvc?rev=1082106&view=rev
Log:
* subversion/libsvn_wc/adm_ops.c
  (new_revert_internal): Don't produce spurious notification about
   execute permissions on Windows.

* subversion/include/private/svn_private_io.h:
  (svn_io__is_finfo_executable): Tweak docstring.

Modified:
    subversion/trunk/subversion/include/private/svn_io_private.h
    subversion/trunk/subversion/libsvn_wc/adm_ops.c

Modified: subversion/trunk/subversion/include/private/svn_io_private.h
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/include/private/svn_io_private.h?rev=1082106&r1=1082105&r2=1082106&view=diff
==============================================================================
--- subversion/trunk/subversion/include/private/svn_io_private.h (original)
+++ subversion/trunk/subversion/include/private/svn_io_private.h Wed Mar 16 
09:49:27 2011
@@ -36,6 +36,8 @@ extern "C" {
 
 /** Set @a *executable TRUE if @a file_info is executable for the
  * user, FALSE otherwise.
+ *
+ * Always returns FALSE on Windows or platforms without user support.
  */
 svn_error_t *
 svn_io__is_finfo_executable(svn_boolean_t *executable,

Modified: subversion/trunk/subversion/libsvn_wc/adm_ops.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/adm_ops.c?rev=1082106&r1=1082105&r2=1082106&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/adm_ops.c (original)
+++ subversion/trunk/subversion/libsvn_wc/adm_ops.c Wed Mar 16 09:49:27 2011
@@ -1430,6 +1430,7 @@ new_revert_internal(svn_wc__db_t *db,
                   notify_required = TRUE;
                 }
 
+#if !defined(WIN32) && !defined(__OS2__)
               if (apr_hash_get(props, SVN_PROP_EXECUTABLE, APR_HASH_KEY_STRING)
                   && !executable)
                 {
@@ -1443,6 +1444,7 @@ new_revert_internal(svn_wc__db_t *db,
                                                      FALSE, scratch_pool));
                   notify_required = TRUE;
                 }
+#endif
             }
         }
     }


Reply via email to