zdctl.py try to fix borked umask handling]

Fred Drake <[email protected]> Thu, 9 Jun 2005 10:44:31 -0400

Newsgroups	gmane.comp.web.zope.coders
Message-ID	<[email protected]>

On 6/9/05, Sidnei da Silva <[email protected]> wrote:
> It doesn't prevent zope from starting at least, so your fix fixes
> that. Haven't tested if it fixes the bug that the previous fix
> intended to fix and ended up breaking zope's startup *wink*.
> 
> From reading the patch it looks very sane, except maybe for the second
> part of it where you put os.umask() inside the if. Shouldn't that one
> be outside the if?

No, it shouldn't be.  Note that it's calling os.umask() twice; the
second call is restoring the old value returned from the first.  The
effect is that we find out the current umask, without changing it
(effectively); this is used only for reporting.

This does warrant a comment; I'll add that and make a tag.  Someone
other than me can deal with Zope 2.8.  :-)

  -Fred

-- 
Fred L. Drake, Jr.    <fdrake at gmail.com>
Zope Corporation