[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1619717 - /subversion/trunk/subversion/libsvn_wc/conflicts.c

From: Stefan Sperling <stsp_at_elego.de>
Date: Fri, 22 Aug 2014 11:40:16 +0100

On Fri, Aug 22, 2014 at 12:26:12PM +0200, Bert Huijben wrote:
> > I'm not sure this should even be flagged as a conflict.
> > However, the victim node kind is wrong.
> I think it should say local file delete, ...
> (but local, left-src and right-src are all victims)
As already mentioned on IRC, when we introduced the term "tree conflict
victim" in 1.6 it always referred to the local node. It never referred
to any incoming ones. I think we should keep this distinction to avoid
misunderstandings. The left-src and right-src cannot be victims.
Whatever the user saw at the affected path in their working tree,
before running the operation, is the victim.

> On a local delete we should really look one layer below the current op-depth
> to see what the original node kind is/was, before the local
> delete/replacement.

I think we should record the victim's node kind when discovering the
conflict, so we won't have to guess it later.
Received on 2014-08-22 12:40:51 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.