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

[patch]: use notifier for wc->wc copy

From: David Kimdon <dwhedon_at_debian.org>
Date: 2003-05-26 23:44:22 CEST


The following patch makes 'svn copy' use a notifier when doing a
working copy to working copy operation. There was a note in the
source indicating that 'copy' didn't need to produce output, so maybe
there is some reason why this patch is not desired (see the diff
below). The note originated in r556, I believe that is before move
printed any output. I took a look in the archives and the log and
didn't get more info as to why this feature would not be desired.

Adding the notifier makes command behavior more consistent. Without
this patch, for example, a 'move' creates output:

$ svn move hi.c hello.c
A hello.c
D hi.c
$ svn status
A + hello.c
D hi.c

While a copy does not:

$ svn copy hi.c hello.c
$ svn status
A + hello.c

With this patch a copy looks like:

$ svn copy hi.c hello.c
A hello.c


* subversion/clients/cmdline/copy-cmd.c (svn_cl__copy) : Do notification
  for working-copy to working-copy 'svn copy' operations.

Index: subversion/clients/cmdline/copy-cmd.c
--- subversion/clients/cmdline/copy-cmd.c (revision 6033)
+++ subversion/clients/cmdline/copy-cmd.c (working copy)
@@ -63,8 +63,12 @@
   dst_is_url = svn_path_is_url (dst_path);
   if ((! src_is_url) && (! dst_is_url))
- /* WC->WC : No notification needed. */
- ;
+ {
+ /* WC->WC */
+ if (! opt_state->quiet)
+ svn_cl__get_notifier (&ctx->notify_func, &ctx->notify_baton,
+ }
   else if ((! src_is_url) && (dst_is_url))
       /* WC->URL : Use notification. */

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon May 26 23:45:10 2003

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.