Re: Typo in documentation for svnsync

On Feb 25, 2007, at 01:53, Steve Axthelm wrote:

> Linked off the current release notes are notes for svnsync:
>
> http://svn.collab.net/repos/svn/trunk/notes/svnsync.txt
>
> On line 23 is this:
>
> if [ "$USER" = "svnsync" ]; then exit 0; fi
>
> which I believe should be:
>
> if [ "$USER" != "svnsync" ]; then exit 0; fi

I dunno, it seems to me that the script is exactly correct as provided:

$ cat <<'EOF' > dest/hooks/pre-revprop-change
#!/bin/sh
USER="$3"

if [ "$USER" = "svnsync" ]; then exit 0; fi

echo "Only the svnsync user can change revprops" >&2
exit 1
EOF

The intention is that if the user is svnsync, then the script should
exit with no error and allow the revprop change to proceed. Any other
user, however, should receive the message that they are not allowed
to make revision property changes. And I believe the script does that
correctly.

-- 
To reply to the mailing list, please use your mailer's Reply To All  
function
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org