| Subject: | [PATCH] Notify.pm sanity check fix |
Somewhere between r3438 and r3447 some additional sanity checking was
added, causing the tool to fail claiming sendmail was not found, before
it looks for it. Attached patch moves the sanity check later down and
fixes the issue.
| Subject: | myfix.diff |
Index: lib/SVN/Notify.pm
===================================================================
--- lib/SVN/Notify.pm (revision 3447)
+++ lib/SVN/Notify.pm (working copy)
@@ -701,8 +701,6 @@
die qq{Missing required "to", "to_regex_map", or "to_email_map" parameter}
unless @{ $params{to} } || $params{to_regex_map}
|| $params{to_email_map};
- die qq{Cannot find sendmail and no "smtp" parameter specified}
- unless $params{sendmail} || $params{smtp};
# Set up default values.
$params{svnlook} ||= $ENV{SVNLOOK} || $class->find_exe('svnlook');
@@ -715,6 +713,9 @@
$params{sendmail} ||= $ENV{SENDMAIL} || $class->find_exe('sendmail')
unless $params{smtp};
+ die qq{Cannot find sendmail and no "smtp" parameter specified}
+ unless $params{sendmail} || $params{smtp};
+
# Set up the environment language.
if ( $params{language} && !$ENV{LANG} ) {
( my $lang_country = $params{language} ) =~ s/_/-/g;