convert_list now runs Utils.ValidEmail() on all addresses, and prints

out a warning if the test fails.
this way comments and most accidental stuff that gets into a file to
import should not be added.
scott

1 files changed, 3 insertions, 1 deletions

diff --git a/bin/convert_list b/bin/convert_list
index a9490e558..532364242 100755
--- a/bin/convert_list
+++ b/bin/convert_list
@@ -107,7 +107,9 @@ def SendExplanation(ml, users):
 def AddMember(ml, addr, digest_p, send_welcome_msg):
     # if the line was empty, just ignore it
     pw = GetRandomPassword()
-
+    if not Mailman.Utils.ValidEmail(addr):
+        print "not a valid email address:", addr
+        return
     try:
         ml.ApprovedAddMember(addr, pw, digest_p, send_welcome_msg)
     except Mailman.Errors.MMAlreadyAMember: