Re: Updating a live website with a post-commit hook script

On Jun 30, 2006, at 08:55, Bob Proulx wrote:

>> open (P, "/usr/bin/svnlook changed $REPOS|");
>> system("/usr/bin/svn up $working_dir/Website/$file");
>
> Lots of hard coded paths! If /usr/bin is not in path then there is a
> problem. Hard coded paths to programs have always come back to bite
> me. I recommend avoiding them.\

Um, since /usr/bin/svnlook is explicitly specified, /usr/bin does not
need to be within the path. This is in fact the whole point, because
in Subversion hooks, the path is not set, so you must use absolute
paths or it won't work. It might be better to define variables (or
constants, depending on language) for the programs used, right at the
top of the script, so that if you need to change the path, they're
all in one place.

The actual problem with the script above was that the svnlook command
is not passed the $REV argument.

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org
Received on Fri Jun 30 12:50:12 2006