svn commit: r1458607 - in /subversion/trunk/subversion: include/svn_dav.h libsvn_ra_serf/options.c mod_dav_svn/version.c

[cm...@apache.org]

Author: cmpilato
Date: Wed Mar 20 01:55:43 2013
New Revision: 1458607

URL: http://svn.apache.org/r1458607
Log:
* subversion/include/svn_dav.h
  (SVN_DAV_NS_DAV_SVN_REVERSE_FILE_REVS): Was
    SVN_DAV_NS_DAV_SVN_GET_FILE_REVS_REVERSE, and now has a value short
    enough to fit into the ridiculous 63-character limitation so kindly
    provided by at least one implementation of Microsoft's Web Folders.

* subversion/mod_dav_svn/version.c
  (get_vsn_options): Track renamed #define.

* subversion/libsvn_ra_serf/options.c
  (capabilities_headers_iterator_callback): Track renamed #define.

Reported by: rhuijben

Modified:
    subversion/trunk/subversion/include/svn_dav.h
    subversion/trunk/subversion/libsvn_ra_serf/options.c
    subversion/trunk/subversion/mod_dav_svn/version.c

Modified: subversion/trunk/subversion/include/svn_dav.h
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/include/svn_dav.h?rev=1458607&r1=1458606&r2=1458607&view=diff
==============================================================================
--- subversion/trunk/subversion/include/svn_dav.h (original)
+++ subversion/trunk/subversion/include/svn_dav.h Wed Mar 20 01:55:43 2013
@@ -277,8 +277,19 @@ extern "C" {
  * @name Custom (extension) values for the DAV header.
  * Note that although these share the SVN_DAV_PROP_NS_DAV namespace
  * prefix, they are not properties; they are header values.
- *
- * @{ **/
+ * @{
+ */
+
+/* ##################################################################
+ * 
+ *    WARNING:  At least some versions of Microsoft's Web Folders
+ *              WebDAV client implementation are unable to handle
+ *              DAV: headers with values longer than 63 characters,
+ *              so please keep these strings within that limit.
+ * 
+ * ##################################################################
+ */
+
 
 /** Presence of this in a DAV header in an OPTIONS request or response
  * indicates that the transmitter supports @c svn_depth_t.
@@ -372,8 +383,8 @@ extern "C" {
  *
  * @since New in 1.8.
  */
-#define SVN_DAV_NS_DAV_SVN_GET_FILE_REVS_REVERSE\
-            SVN_DAV_PROP_NS_DAV "svn/get-file-revs-reverse"
+#define SVN_DAV_NS_DAV_SVN_REVERSE_FILE_REVS\
+            SVN_DAV_PROP_NS_DAV "svn/reverse-file-revs"
 
 
 /** @} */

Modified: subversion/trunk/subversion/libsvn_ra_serf/options.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/options.c?rev=1458607&r1=1458606&r2=1458607&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/options.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/options.c Wed Mar 20 01:55:43 2013
@@ -203,7 +203,7 @@ capabilities_headers_iterator_callback(v
           svn_hash_sets(session->capabilities,
                         SVN_RA_CAPABILITY_INHERITED_PROPS, capability_yes);
         }
-      if (svn_cstring_match_list(SVN_DAV_NS_DAV_SVN_GET_FILE_REVS_REVERSE,
+      if (svn_cstring_match_list(SVN_DAV_NS_DAV_SVN_REVERSE_FILE_REVS,
                                  vals))
         {
           svn_hash_sets(session->capabilities,

Modified: subversion/trunk/subversion/mod_dav_svn/version.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/version.c?rev=1458607&r1=1458606&r2=1458607&view=diff
==============================================================================
--- subversion/trunk/subversion/mod_dav_svn/version.c (original)
+++ subversion/trunk/subversion/mod_dav_svn/version.c Wed Mar 20 01:55:43 2013
@@ -150,7 +150,7 @@ get_vsn_options(apr_pool_t *p, apr_text_
   apr_text_append(p, phdr, SVN_DAV_NS_DAV_SVN_PARTIAL_REPLAY);
   apr_text_append(p, phdr, SVN_DAV_NS_DAV_SVN_INHERITED_PROPS);
   apr_text_append(p, phdr, SVN_DAV_NS_DAV_SVN_INLINE_PROPS);
-  apr_text_append(p, phdr, SVN_DAV_NS_DAV_SVN_GET_FILE_REVS_REVERSE);
+  apr_text_append(p, phdr, SVN_DAV_NS_DAV_SVN_REVERSE_FILE_REVS);
   /* Mergeinfo is a special case: here we merely say that the server
    * knows how to handle mergeinfo -- whether the repository does too
    * is a separate matter.