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