On Mon, Oct 03, 2011 at 02:59:25PM +0200, Johan Corveleyn wrote:
> On Mon, Oct 3, 2011 at 2:35 PM, Johan Corveleyn <jcorvel_at_gmail.com> wrote:
> > On Mon, Oct 3, 2011 at 2:16 PM, Kyle Leber <kyle.leber_at_gmail.com> wrote:
> >> I set the mime-type to "application/octet-stream" in the working copy prior
> >> to merge and this fixed the problem. No more heavy CPU usage or excessive
> >> time spent on the file.
> >
> > I'm glad it helped. Apart from the performance, it's important that
> > svn does this merge the "binary way", because as you said line-based
> > merges are not correct for this file.
>
> It may also interest you (and other readers of this thread) that there
> is an open enhancement request for making text-merges take the same
> shortcut as binary-merges (if mine == merge-left then set merged :=
> merge-right), to avoid expensive diffing [1]. But that hasn't been
> addressed yet.
>
>
> [1] http://subversion.tigris.org/issues/show_bug.cgi?id=4009 : Big
> trivial text files merged MUCH slower than binary - pls optimize.
>
I think we should also file an issue about the problem discussed
in this thread. svn should take properties on the left/right side of the
merge into account when determining whether to treat a file as binary.
I guess it should run the binary merge if any of left, right, or the
target are marked as binary.
Received on 2011-10-03 15:04:49 CEST