Hi Prabhu,
I understand your patch fixes the following two cases.
1. svn di -cN explicitly_reinstated_file_with_mod_at_rN
Without your patch we get a failure
"explicitly_reinstated_file_with_mod_at_rN does not exist at rN-1"
With your patch we get *modification* alone.
2. svn di -cN Parent_Dir_of_explicitly_reinstated_file_at_rN
Without your patch we get all lines as added.
With your patch we get only what got injected along with copy in rN.
So if somebody wants all lines as added in above cases they can use
--show-copies-as-adds.
Please fix your patch to remove this new param diff-copy-from in all the
layers and assume it to be TRUE unless "--show-copies-as-adds" is given.
Thanks
With regards
Kamesh Jayachandran
On 11/30/2010 03:26 PM, Prabhu Gnana Sundar wrote:
> Hi,
>
> In my previous patch [1] I suggested the addition of the
> '--diff-copy-from' switch to 'svn diff' to solve the cases where 'svn
> diff' would do the diff against the copyfrom source file when a file is
> reinstated or diff target is *not* explicit file.
>
> But since there is already a point that diff behaves differently in
> different scenarios, making 'svn diff' work the same in all scenarios is
> one good option. This patch would solve the problem of making 'svn diff'
> work the same in all conditions by passing 'TRUE' for the
> 'diff_copy_from' option by default. Hence I removed the
> '--diff-copy-from' switch in this patch. This is a more generic way to
> bridge the gap.
>
> I am attaching the patch and the log with this mail. Please review and
> comment on the same.
>
> I still retain this param on svn_client_diff5 and svn_ra_diff4 in case
> somebody wants old behaviour, otherwise I can remove that too.
>
> Regards
> Prabhu
>
> [1]http://mail-archives.apache.org/mod_mbox/subversion-dev/201011.mbox/%
> 3C1291018400.4021.3.camel_at_prabhugnanasundar%3E
Received on 2010-12-01 16:15:05 CET