Re: revprop change notifications (was: svn commit: rev 5289 ...)

Greg Stein <gstein@lyra.org> writes:

> I thought the original idea was that the previous content would be given to
> the script on stdin, but that we just hadn't implemented that yet. We could
> also provide the new content to the pre- hook in case it wanted to perform a
> check on the contents before allowing it (e.g. validate the format/schema of
> the property).

That was indeed the plan.

> (a tempfile won't work since it has to have a unique name, and there is no
> way to transmit that name to the post-change)

?

I know we weren't planning to use a tempfile, but why exactly would
this not work, again?

    tmp_dir = (the not yet existent)apr_dir_temp_get();
    path = svn_io_open_unique_file(tmp_dir, "svn-ps", "tmp")
    write(path, old_value)

Pass PATH to the post-revprop-change hook as a cmd-line arg (with
REPOS, PROPNAME, etc). The hook reads from PATH, does its thing, and
then the repos layer unlinks PATH.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Mar 14 05:18:33 2003