diff hgext/patchbomb.py @ 4565:1cf908c00479

Don't validate email config if we're not sending email.
author Bryan O'Sullivan <bos@serpentine.com>
date Tue, 12 Jun 2007 14:44:47 -0700
parents d48e1b5f8265
children 087b3ae4f08a
line wrap: on
line diff
--- a/hgext/patchbomb.py	Tue Jun 12 14:39:36 2007 -0700
+++ b/hgext/patchbomb.py	Tue Jun 12 14:44:47 2007 -0700
@@ -223,7 +223,9 @@
                 pass
             os.rmdir(tmpdir)
 
-    if not opts['test']:
+    really_sending = not (opts['test'] or opts['mbox'])
+
+    if really_sending:
         mail.validateconfig(ui)
 
     if not (revs or opts.get('rev') or opts.get('outgoing')):
@@ -351,7 +353,7 @@
 
     ui.write('\n')
 
-    if not opts['test'] and not opts['mbox']:
+    if really_sending:
         mailer = mail.connect(ui)
     parent = None