I've added my thoughts to the issue ; essentially, I've observed that
svn 1.6 stores tree_conflict_data="(foo bar)" where 1.7 stores
tree_conflict_data="foo bar", and this patch makes 'svn upgrade'
I haven't tested upgrading from interim 1.7-dev formats, and I don't
recall offhand at which point we made the transition to lose the extra
level of ().
Reviews welcome, otherwise I'll commit it in a few days.
Fix issue #3840, "empty conflict description in db after update from 1.6".
(write_entry): Strip one level of skel-wrapping () from TREE_CONFLICT_DATA
when upgrading entries.
--- subversion/libsvn_wc/entries.c (revision 1085641)
+++ subversion/libsvn_wc/entries.c (working copy)
@@ -1824,8 +1824,20 @@ write_entry(struct write_baton **entry_node,
+ /* Issue #3840: 1.6 uses one pair of () more than we do, so
+ strip it. Thus, "()" becomes NULL and "((skel1) (skel2))"
+ becomes "(skel1) (skel2)". */
+ svn_skel_t *skel;
actual_node = MAYBE_ALLOC(actual_node, scratch_pool);
- actual_node->tree_conflict_data = entry->tree_conflict_data;
+ skel = svn_skel__parse(entry->tree_conflict_data,
+ if (skel->children)
+ actual_node->tree_conflict_data = svn_skel__unparse(skel->children,
+ actual_node->tree_conflict_data = NULL;
if (entry->file_external_path != NULL)
Alan Wood wrote on Mon, Mar 21, 2011 at 23:52:20 +1300:
> On 21 Mar 2011 at 10:48, Stefan Sperling wrote:
> > On Mon, Mar 21, 2011 at 12:33:34PM +1300, Alan Wood wrote:
> > > There should not have been any conflict data of any sort in this
> > > working copy as I am the only developer and I have only every had
> > > one working copy.
> > That's not a valid assumption.
> > It is possible to inflict tree conflicts upon yourself, with only
> > one working copy: http://subversion.apache.org/faq.html#self-tree-conflict
> Thanks for the link. I had read this a while ago, but forgotton all about it.
> > > I hope someone may be able to plug this little hole in the upgrade
> > > system.
> > Looks like the upgrade code shouldn't insert empty conflict description
> > skels into the DB. Can you file an issue that points to this mailing list
> > thread, so that this doesn't fall through the cracks? Thanks.
> Filed issue# 3840
> Sorry for the confusion with the version labels.
> > And thanks for testing!
> No trouble, would like to have more time for it.
> Alan Wood
> New Zealand
> Phone +64 6 835 4505
Received on 2011-03-26 05:42:09 CET