Daniel Rall wrote:
> On Tue, 03 Oct 2006, Garrett Rooney wrote:
>
>> On 10/2/06, Daniel Rall <dlr@collab.net> wrote:
>>
>>> My desired behavior for 'svnadmin setrevprop' is be:
>>>
>>> o Bypass revprop change hook scripts by default, as this is a program
>>> intended for repository administration. This behavior differs from
>>> that of 'svnadmin setlog'.
>>>
>>> o Allow hook scripts to be invoked using command-line flags, a la
>>> 'svnadmin load':
>>> --use-pre-revprop-change-hook
>>> --use-post-revprop-change-hook
>>>
>>> I'd be happy to implement the above.
>> Perhaps something like --skip-hook-scripts with a default of skipping
>> all of them, but optionally taking a list of hooks to skip? Or the
>> other way around with --run-hook-scripts. Honestly I'm fine with it
>> the way it is though.
> ...
>
> --skip-hook-scripts is what the existing --bypass-hooks flag does
> (which Mike doesn't like). Running hook scripts does not strike me as
> the desired default behavior for an administrative program like
> 'svnadmin', nor would it be consistent with the 'load' sub-command
> (though that difference is arguably understandable based on the
> command core use cases). The main reason to make --bypass-hooks a
> flag instead of a default is to help administrators unfamiliar with
> 'svnadmin' from accidentally shooting themselves in the foot.
>
> The only hook scripts we run for a 'propset --revprop' are
> pre-/post-revprop-change hooks. Do we really need a list here? That
> interface seems less friendly than two well-described, well-named
> flags.
I think a list is overkill. And I like switching the default to be to
skip the hooks. Let's add --use-(pre|post)-revprop-change hook. Doing
so means we don't have to validate an option parameters (these are
boolean options), and gives us consistency with the 'svnadmin load' hook
interaction.
--
C. Michael Pilato <cmpilato@collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on Tue Oct 3 19:52:45 2006