diff options
| author | Gervase Markham <gerv@gerv.net> | 2015-08-07 11:13:20 +0100 |
|---|---|---|
| committer | Gervase Markham <gerv@gerv.net> | 2015-08-07 11:13:20 +0100 |
| commit | 2d2c85f63dbde9b49079a0ab0c8c5ddcda33acec (patch) | |
| tree | 23c6bed69f7543c086716dc6c52e32f95267e2a1 /Bugzilla/Config | |
| parent | 29ee9d3661b37de486360c634aab099833064915 (diff) | |
| download | bugs-2d2c85f63dbde9b49079a0ab0c8c5ddcda33acec.tar bugs-2d2c85f63dbde9b49079a0ab0c8c5ddcda33acec.tar.gz bugs-2d2c85f63dbde9b49079a0ab0c8c5ddcda33acec.tar.bz2 bugs-2d2c85f63dbde9b49079a0ab0c8c5ddcda33acec.tar.xz bugs-2d2c85f63dbde9b49079a0ab0c8c5ddcda33acec.zip | |
Bug 1191255 - Remove requirement to specify SMTP server during installation on Windows. r=LpSolit, a=simon
Diffstat (limited to 'Bugzilla/Config')
| -rw-r--r-- | Bugzilla/Config/Common.pm | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/Bugzilla/Config/Common.pm b/Bugzilla/Config/Common.pm index 414894773..a5b32c1db 100644 --- a/Bugzilla/Config/Common.pm +++ b/Bugzilla/Config/Common.pm @@ -299,7 +299,7 @@ sub check_mail_delivery_method { my $check = check_multi(@_); return $check if $check; my $mailer = shift; - if ($mailer eq 'sendmail' and ON_WINDOWS) { + if ($mailer eq 'Sendmail' and ON_WINDOWS) { # look for sendmail.exe return "Failed to locate " . SENDMAIL_EXE unless -e SENDMAIL_EXE; @@ -346,6 +346,8 @@ sub check_smtp_server { my $host = shift; my $port; + return '' unless $host; + if ($host =~ /:/) { ($host, $port) = split(/:/, $host, 2); unless ($port && detaint_natural($port)) { |