[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-issues
Subject:    [Issue 4408] New - svn diff --summarize -c N produces invalid output against dirs added in rN.
From:       breser () tigris ! org
Date:       2013-08-09 5:09:49
Message-ID: iz4408 () subversion ! tigris ! org
[Download RAW message or body]

http://subversion.tigris.org/issues/show_bug.cgi?id=4408
                 Issue #|4408
                 Summary|svn diff --summarize -c N produces invalid output agai
                        |nst dirs added in rN.
               Component|subversion
                 Version|1.7.x
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|libsvn_client
             Assigned to|issues@subversion
             Reported by|breser






------- Additional comments from breser@tigris.org Thu Aug  8 22:09:49 -0700 2013 \
------- $ svnadmin create repo
$ svn co file://`pwd`/repo wc
Checked out revision 0.
$ svn mkdir wc/trunk
A         wc/trunk
$ svn ci wc -mm
Adding         wc/trunk

Committed revision 1.
$ svn diff --summarize file://`pwd`/repo -c 1
A       file:///Users/breser/diff-summarize/repo/trunk
$ svn diff --summarize file://`pwd`/repo/trunk -c 1
A       file:///Users/breser/diff-summarize/repo/trunk/trunk

Note that trunk is repeated on the second summarize command.  This is because the \
trunk does not exist in r0 so the editor drive is  the rooted at the root of the \
repository.  As such the reported path is "trunk" and the code in the \
summarize_function combines the  URL with the  reported path and trunk gets repeated.

This is related to issue #2765 which is the same problem except for a file.  However \
at the time that bug was fixed this problem was  not possible because the requested \
path had to exist in both revisions.  Issue #2873 solved that and allowed this \
situation to exist.

1.8.x and newer are not impacted because when diff --summarize was rewritten to \
function with not just two repos paths this issue  was fixed as a side effect.

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=463&dsMessageId=3062318

To unsubscribe from this discussion, e-mail: \
[issues-unsubscribe@subversion.tigris.org].


[prev in list] [next in list] [prev in thread] [next in thread]