...

...

...

...

"DM" == Dan Mick Dan.Mick@West.Sun.COM writes:

DM> Here's a fix that's probably safe, and fixes the "newlist"
DM> problem.  Of course I'm not sure if it's the right final fix,
DM> but I figure while Barry's examining the problem, this might
DM> be worth a try for some of you.

I just got back from a gig -- thanks for the quick fix Dan. Here's a slightly better one that also fixes a few other ugly bits.

This is serious enough to warrant a quick beta4, but I want to spend a couple of hours looking into Ricardo's problem (which I still can't reproduce). Better that then having to do a beta5 over the weekend ;} I'm just too beat tonight.

-Barry

Index: MailList.py

RCS file: /cvsroot/mailman/mailman/Mailman/MailList.py,v retrieving revision 1.172 diff -u -r1.172 MailList.py --- MailList.py 2000/06/28 18:40:48 1.172 +++ MailList.py 2000/06/30 03:04:42 @@ -782,23 +782,21 @@ Utils.MakeDirTree(os.path.join(mm_cfg.LIST_DATA_DIR, name)) self._full_path = os.path.join(mm_cfg.LIST_DATA_DIR, name) self._internal_name = name

• self.Lock()
• self.InitVars(name, admin, crypted_password)
• self._ready = 1
• self.InitTemplates()
• self.Save()
• self.CreateFiles()
• def CreateFiles(self):

•    # Don't use Lock() since that tries to load the non-existant config.db
  

•    self.__lock.lock()
  

•    self.InitVars(name, admin, crypted_password)
  

•    self._ready = 1
  

•    self.InitTemplates()
  

•    self.Save()
  

  # Touch these files so they have the right dir perms no matter what. # A "just-in-case" thing. This shouldn't have to be here. ou = os.umask(002) try:

•    open(os.path.join(mm_cfg.LOCK_DIR, '%s.lock' % 
  

• 	      self._internal_name), 'a+').close()
  

•    open(os.path.join(self._full_path, "next-digest"), "a+").close()
  

•    open(os.path.join(self._full_path, "next-digest-topics"),
  

•  "a+").close()
  

•        path = os.path.join(self._full_path, 'next-digest')
  

•        fp = open(path, "a+")
  

•        fp.close()
  

•    fp = open(path+'-topics', "a+")
  

•        fp.close()
  

  finally: os.umask(ou)