do not set smtp host by default - raise error instead
authorVadim Gelfer <vadim.gelfer@gmail.com>
Mon, 15 May 2006 08:47:42 -0700
changeset 2286 9f745d3675d4
parent 2285 0912f807b7ff
child 2287 3f18d1eea370
do not set smtp host by default - raise error instead
mercurial/ui.py
--- a/mercurial/ui.py	Sun May 14 23:49:31 2006 -0700
+++ b/mercurial/ui.py	Mon May 15 08:47:42 2006 -0700
@@ -271,8 +271,10 @@
 
     def sendmail(self):
         s = smtplib.SMTP()
-        s.connect(host = self.config('smtp', 'host', 'mail'),
-                  port = int(self.config('smtp', 'port', 25)))
+        mailhost = self.config('smtp', 'host')
+        if not mailhost:
+            raise util.Abort(_('no [smtp]host in hgrc - cannot send mail'))
+        s.connect(host=mailhost, port=int(self.config('smtp', 'port', 25)))
         if self.configbool('smtp', 'tls'):
             s.ehlo()
             s.starttls()