Message185777
In the mail hook, there's a comment that says:
'''
To set the SMTP server to something other than localhost, add a [smtp]
section to your hgrc:
[smtp]
host = mail.python.org
port = 25
'''
This is not true. The default host is '', which tells smtplib.SMTP to not make an initial connection. If we want the comment to be correct, then we need to set 'localhost' as the default:
- host = ui.config('smtp', 'host', '')
+ host = ui.config('smtp', 'host', 'localhost')
I guess there's some chance that we want to force the smtp host to be set. If that's the case then then comment should be changed and the code modified to require the host be specified in the config file (or at least a better error message than 'please run connect() first').
Adding Antoine because annotate points to him for these lines. |
|
Date |
User |
Action |
Args |
2013-04-01 22:00:55 | eric.smith | set | recipients:
+ eric.smith, pitrou |
2013-04-01 22:00:55 | eric.smith | set | messageid: <1364853655.88.0.718145768613.issue17612@psf.upfronthosting.co.za> |
2013-04-01 22:00:55 | eric.smith | link | issue17612 messages |
2013-04-01 22:00:55 | eric.smith | create | |
|