diff options
| author | bwarsaw | 2001-08-20 15:05:44 +0000 |
|---|---|---|
| committer | bwarsaw | 2001-08-20 15:05:44 +0000 |
| commit | f7cd2f45dacbaa975be6d3a03176c9b02b24d444 (patch) | |
| tree | a1328e980ff5e7bcac1d03f66bb6ca879cca5373 /Mailman/MailList.py | |
| parent | c9690a16f93f7b510657feecd91f40d0c0273602 (diff) | |
| download | mailman-f7cd2f45dacbaa975be6d3a03176c9b02b24d444.tar.gz mailman-f7cd2f45dacbaa975be6d3a03176c9b02b24d444.tar.zst mailman-f7cd2f45dacbaa975be6d3a03176c9b02b24d444.zip | |
Create(): Let any exceptions in the os.makedirs() call percolate up.
I.e. don't catch and transform OSErrors to MMUnknownListErrors.
Also, don't do a Save() here. The caller is now responsible for
saving the list.
Diffstat (limited to 'Mailman/MailList.py')
| -rw-r--r-- | Mailman/MailList.py | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/Mailman/MailList.py b/Mailman/MailList.py index 42ff66613..1b9e24bf7 100644 --- a/Mailman/MailList.py +++ b/Mailman/MailList.py @@ -373,10 +373,7 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin, Utils.ValidateEmail(admin) omask = os.umask(0) try: - try: - os.makedirs(os.path.join(mm_cfg.LIST_DATA_DIR, name), 02775) - except OSError: - raise Errors.MMUnknownListError + os.makedirs(os.path.join(mm_cfg.LIST_DATA_DIR, name), 02775) finally: os.umask(omask) self._full_path = os.path.join(mm_cfg.LIST_DATA_DIR, name) @@ -389,7 +386,7 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin, self.available_languages = [self.preferred_language] else: self.available_languages = langs - self.Save() + # |