Re: [PATCH] Setting email in hook scripts

Gustavo Niemeyer wrote:
>
> > What would you say about re-submitting using -f (for "from address")
> > instead of -e? "email" is quite ambiguous.
>
> You're right. Here is an updated patch. Thank you!

Minor nit. Can you keep the options in alphabetical order, in the
lines:

> + if (my ($opt) = $arg =~ /^-([hflmrs])/)
> my %opt_to_hash_key = (h => 'hostname',
> + f => 'from_addr',
> l => 'log_file',

Also, a log message as described in HACKING would be useful.

Best,
Blair

-- 
Blair Zajac <blair@orcaware.com>
Plots of your system's performance - http://www.orcaware.com/orca/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org