use os.path.isdir instead of catching error

datactive · May 4, 2021 · aed908f · aed908f
1 parent f9c01eb
commit aed908f
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/bigbang/listserv.py b/bigbang/listserv.py
@@ -987,10 +987,8 @@ def from_mailing_lists(
                 if len(mlist) != 0:
                     if instant_save:
                         dir_out = CONFIG.mail_path + name
-                        try:
+                        if os.path.isdir(dir_out) is False:
                             os.mkdir(dir_out)
-                        except FileExistsError:
-                            pass  # temporary directory already exists, that's cool
                         mlist.to_mbox(dir_out=dir_out)
                     else:
                         logger.info(f"Recorded the list {mlist.name}.")
@@ -1087,10 +1085,8 @@ def get_lists_from_url(
                     if len(mlist) != 0:
                         if instant_save:
                             dir_out = CONFIG.mail_path + name
-                            try:
+                            if os.path.isdir(dir_out) is False:
                                 os.mkdir(dir_out)
-                            except FileExistsError:
-                                pass  # temporary directory already exists, that's cool
                             mlist.to_mbox(dir_out=CONFIG.mail_path)
                             archive.append(mlist.name)
                         else: