fixing issue 679

Hello

I added some comments to issue 679 but although they appear on the
issue they don't appear to have made it to the mailing list, nor to
the mailing list web archive :-( I'll post them here:

Looking at libsvn_client/commit_util.c:226 harvest_commitables() there
seems to be some code to handle this, but it's disabled.

  /* Check for the copied-subtree addition case. */
  if ((entry->copied || copy_mode)
      && (entry->schedule == svn_wc_schedule_normal))
    {
#if 0 /* ### todo: Find a better way to do this that doesn't require
         reading the parent's entry. */
      svn_revnum_t p_rev = entry->revision - 1; /* arbitrary non-equal
value */
      svn_boolean_t wc_root = FALSE;

Is reading the parent's entry bad? Is it bad enough to warrant making
'svn cp wc_dir wc_dir' break? Or is this code disabled by mistake?

Is reading the parent's entry bad simply from an efficiency point of
view? If that is the case I think adding an svn_entry_t *parent to
the function would be quite straight-forward.

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