subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From svn-r...@apache.org
Subject svn commit: r1356567 - in /subversion/branches/1.7.x: ./ STATUS subversion/libsvn_repos/replay.c
Date Tue, 03 Jul 2012 04:01:32 GMT
Author: svn-role
Date: Tue Jul  3 04:01:31 2012
New Revision: 1356567

URL: http://svn.apache.org/viewvc?rev=1356567&view=rev
Log:
Merge the r1338810 group from trunk:

 * r1338810, r1339164
   Fix issue #4184 ("partial svnsync drops properties when converting
   copies to adds").
   Justification:
     Syncs that don't aren't synced at all.
   Notes:
     r1338810 taught the code to send properties for demoted copies.
     r1339164 fixed a theoretical use of uninitialized memory.
   Votes:
     +1: cmpilato, rhuijben, philip

Modified:
    subversion/branches/1.7.x/   (props changed)
    subversion/branches/1.7.x/STATUS
    subversion/branches/1.7.x/subversion/libsvn_repos/replay.c

Propchange: subversion/branches/1.7.x/
------------------------------------------------------------------------------
  Merged /subversion/trunk:r1338810,1339164

Modified: subversion/branches/1.7.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x/STATUS?rev=1356567&r1=1356566&r2=1356567&view=diff
==============================================================================
--- subversion/branches/1.7.x/STATUS (original)
+++ subversion/branches/1.7.x/STATUS Tue Jul  3 04:01:31 2012
@@ -93,14 +93,3 @@ Veto-blocked changes:
 
 Approved changes:
 =================
-
- * r1338810, r1339164
-   Fix issue #4184 ("partial svnsync drops properties when converting
-   copies to adds").
-   Justification:
-     Syncs that don't aren't synced at all.
-   Notes:
-     r1338810 taught the code to send properties for demoted copies.
-     r1339164 fixed a theoretical use of uninitialized memory.
-   Votes:
-     +1: cmpilato, rhuijben, philip

Modified: subversion/branches/1.7.x/subversion/libsvn_repos/replay.c
URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x/subversion/libsvn_repos/replay.c?rev=1356567&r1=1356566&r2=1356567&view=diff
==============================================================================
--- subversion/branches/1.7.x/subversion/libsvn_repos/replay.c (original)
+++ subversion/branches/1.7.x/subversion/libsvn_repos/replay.c Tue Jul  3 04:01:31 2012
@@ -693,10 +693,18 @@ path_driver_cb_func(void **dir_baton,
         }
     }
 
-  /* Handle property modifications. */
   if (! do_delete || do_add)
     {
-      if (change->prop_mod)
+      /* Is this a copy that was downgraded to a raw add?  (If so,
+         we'll need to transmit properties and file contents and such
+         for it regardless of what the CHANGE structure's text_mod
+         and prop_mod flags say.)  */
+      svn_boolean_t downgraded_copy = (change->copyfrom_known
+                                       && change->copyfrom_path
+                                       && (! copyfrom_path));
+
+      /* Handle property modifications. */
+      if (change->prop_mod || downgraded_copy)
         {
           if (cb->compare_root)
             {
@@ -740,14 +748,9 @@ path_driver_cb_func(void **dir_baton,
             }
         }
 
-      /* Handle textual modifications.
-
-         Note that this needs to happen in the "copy from a file we
-         aren't allowed to see" case since otherwise the caller will
-         have no way to actually get the new file's contents, which
-         they are apparently allowed to see. */
+      /* Handle textual modifications. */
       if (change->node_kind == svn_node_file
-          && (change->text_mod || (change->copyfrom_path && ! copyfrom_path)))
+          && (change->text_mod || downgraded_copy))
         {
           svn_txdelta_window_handler_t delta_handler;
           void *delta_handler_baton;



Mime
View raw message