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

[PATCH] svnmirror nothing to do case

From: Martin Furter <mf_at_rola.ch>
Date: 2004-12-22 00:50:53 CET

Hello

Would a kind soul please commit this patch by John Belmonte ?

Thanks
Martin

<<<

Update to svnmirror.sh submitted by John Belmonte <john@neggie.net>.

* contrib/server-side/svnmirror.sh
    + print a non-error message to stdout instead of stderr.

>>>

Index: contrib/server-side/svnmirror.sh
===================================================================
--- contrib/server-side/svnmirror.sh (revision 12487)
+++ contrib/server-side/svnmirror.sh (working copy)
@@ -50,6 +50,8 @@
 # - for some help with the svn commands
 # - Bjørn Magnus Mathisen <epic@generation.no>
 # - for pointing out i forgot to replace one ssh with $LSSH
+# - John Belmonte <john@neggie.net>
+# - for pointing out that we use stderr for a non-error message
 #
 # Users:
 # our biggest users atm are Mono Project and MonoDevelop
@@ -74,6 +76,8 @@
 #
 # Changes:
 #
+# 0.0.6
+# + print a non-error message to stdout instead of stderr.
 # 0.0.5
 # + added DUMPPARAMS to specify the params for the svnadmin dump call
 # + added note about svnadmin 1.1 and "--deltas" cmdline option
@@ -330,7 +334,7 @@
 #
 if [ ${RVERSION} -eq ${LVERSION} ] ; then
     [ ${VERBOSE} = true ] && \
- echo "both repositories are already at ${LVERSION}" >&2
+ echo "both repositories are already at ${LVERSION}"
     exit 0
 fi

On Tue, 21 Dec 2004, John Belmonte wrote:

> Hello,
>
> Regarding svnmirror, if the repositories are in sync, I don't think
> there should be a message on stderr. This allows svnmirror to be used
> from a cron job easily, where normal output is logged to a file, and
> anything on stderr propagates to cron as an error.
>
> If I should be submitting this patch elsewhere, please let me know.
>
> Regards,
> -John
>
>
>
> --- svnmirror~ 2004-12-21 12:11:39.000000000 -0500
> +++ svnmirror 2004-12-21 13:39:59.000000000 -0500
> @@ -330,7 +330,7 @@
> #
> if [ ${RVERSION} -eq ${LVERSION} ] ; then
> [ ${VERBOSE} = true ] && \
> - echo "both repositories are already at ${LVERSION}" >&2
> + echo "both repositories are already at ${LVERSION}"
> exit 0
> fi
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Dec 22 00:54:24 2004

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.