Preston-Landers / concurrent-log-handler

fork of ConcurrentLogHandler

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

parameter owner in windows raise error

wshyjt opened this issue · comments

commented

init.py line 237

        if owner and os.chown and pwd and grp:
            # noinspection PyUnresolvedReferences
            self._set_uid = pwd.getpwnam(self.owner[0]).pw_uid
            # noinspection PyUnresolvedReferences
            self._set_gid = grp.getgrnam(self.owner[1]).gr_gid

wndows system there has't os.chown, run this code raise error.
line 73

try:
    import pwd
    import grp
except ImportError:
    pwd = grp = None

windows cannot import pwd and grp , so pwd = grp = None
so init.py line 237 change pwd and grp before os.chown ,ignore os.chown

        if owner and pwd and grp and os.chown:
            # noinspection PyUnresolvedReferences
            self._set_uid = pwd.getpwnam(self.owner[0]).pw_uid
            # noinspection PyUnresolvedReferences
            self._set_gid = grp.getgrnam(self.owner[1]).gr_gid

OK, thanks for the report. I'll check it soon.