You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by kf...@collab.net on 2004/05/24 14:49:43 UTC
Re: svn commit: r9855 - in trunk/subversion: libsvn_repos libsvn_wc
ghudson@tigris.org writes:
> Log:
> Fix two bugs in the handling of incomplete subdirectories. (There is
> arguably a third bug in the update editor, in that it doesn't clear
> the incomplete flag on a subdir if the driver doesn't open it. But
> that bug is hard to fix, and it is currently masked because the driver
> will open an incomplete subdir even if there are no edits to make to
> it because it is empty.)
Is that parenthetical aside captured anywhere in the issue tracker or
as a comment in the code? Should it be, perhaps?
-Karl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: r9855 - in trunk/subversion: libsvn_repos libsvn_wc
Posted by Greg Hudson <gh...@MIT.EDU>.
On Mon, 2004-05-24 at 10:49, kfogel@collab.net wrote:
> ghudson@tigris.org writes:
> > Log:
> > Fix two bugs in the handling of incomplete subdirectories. (There is
> > arguably a third bug in the update editor, in that it doesn't clear
> > the incomplete flag on a subdir if the driver doesn't open it. But
> > that bug is hard to fix, and it is currently masked because the driver
> > will open an incomplete subdir even if there are no edits to make to
> > it because it is empty.)
> Is that parenthetical aside captured anywhere in the issue tracker or
> as a comment in the code? Should it be, perhaps?
No, I don't think so.
It could conceivably become a problem if the libsvn_repos driver learns
to prune when it sees that the source and target directory are unrelated
but empty. (Why we would ever implement that optimization is unclear;
you want to prune off big chunks, not empties. Perhaps it could be a
consequence of a more comprehensive optimization.) The fix would seem
to involve a final walk of the working copy target, clearing incomplete
flags, after a successful update.
So, I guess I'll file a low-priority issue for it.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org