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

List:       subversion-issues
Subject:    =?UTF-8?B?W0lzc3VlIDIyMzNdIE5ldyAtIFNob3cgb25seSBjaGFuZ2VzIG9mIA==?=
From:       slamb () tigris ! org
Date:       2005-02-25 18:41:11
Message-ID: 20050225184111.13844.qmail () tigris ! org
[Download RAW message or body]

http://subversion.tigris.org/issues/show_bug.cgi?id=2233
                  Issue #:|2233
                  Summary:|Show only changes of copied files in "svnlook diff"
                Component:|subversion
                  Version:|all
                 Platform:|All
                      URL:|
               OS/Version:|All
                   Status:|NEW
        Status whiteboard:|
                 Keywords:|
               Resolution:|
               Issue type:|DEFECT
                 Priority:|P3
             Subcomponent:|svnlook
              Assigned to:|issues@subversion
              Reported by:|slamb






------- Additional comments from slamb@tigris.org Fri Feb 25 10:41:11 -0800 2005 \
------- (I'm using Subversion 1.0.9 on this server, but I believe this behavior has \
not changed since.)

Currently, svnlook diff shows every line as added on a copied file. It displays the \
headers I would  expect:

Copied: NEWPATH (from rev OLDREV, OLDPATH)
===============================================================
====
--- OLDPATH        OLDDATE (rev OLDREV)
+++ NEWPATH        NEWDATE (rev NEWREV)

but the actual diff is not consistent with that. Rather than showing the changes \
between  OLDPATH@OLDREV and NEWPATH@NEWREV, it shows the differences between an empty \
file and  NEWPATH@NEWREV.

This is quite odd, especially since the diff output is skipped entirely if the file \
is not changed. It seems  the intent was to show changes in the file as it was \
copied, but that's not what it does.

I'd like to see the diff reflect the header. I get HTMLized color diff emails with \
SVN::Notify, which uses  "svnlook diff" internally. It'd be more useful to me to see \
the changes in the copied file, as mailer.py  does.

Otherwise, the header should be changed to be accurate. Maybe there could be a \
commandline  argument to switch between the two, if others find this behavior more \
useful.

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@subversion.tigris.org
For additional commands, e-mail: issues-help@subversion.tigris.org


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