On 8/5/06, Nico Kadel-Garcia <nkadel@comcast.net> wrote:
> I wrote about this before: the "svnadmin hotcopy" tool does not successfully
> replicate symlinks in the repositories.
>
> Someone sent me a nice patch for the "repos.c" file, which added the
> capability of duplicating symlinks, but I also had to add a patch to io.c in
> order to actually have the function called as the walk through the
> directories lists files and directories to move.
>
> I'm attaching a copy of the patch against the current trunk for subversion,
> I'll also include it in the issue tracker page I created.
The patch seems mostly reasonable, but the bit where you fprintf to
stderr in the case of something you don't understand is just wrong.
We can't just spew stuff to stderr in a library function like that,
there's no way to tell if anyone's even looking at stderr, this could
be embedded in a GUI app or something. If we NEED a way to warn
without erroring, we have to do it via some sort of callback/baton
pair, otherwise treat it as an error and return an svn_error_t just
like any other problem. Or just look at the other potential cases and
figure out if they need to be handled, if not just ignore them.
-garrett
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org
Received on Mon Aug 7 17:23:39 2006