Author vstinner
Recipients eric.araujo, eric.smith, giampaolo.rodola, gregory.p.smith, ncoghlan, neologix, petri.lehtinen, sandro.tosi, vstinner
Date 2011-06-19.22:57:14
SpamBayes Score 5.53583e-05
Marked as misclassified No
Message-id <1308524235.63.0.345095182484.issue12191@psf.upfronthosting.co.za>
In-reply-to
Content
shutil_chown-default-v3.patch:
 - os.chown() is only available on Unix, shutil.chown() should not be defined if os.chown() doesn't exist (require to add a @unittest.skipUnless decorator to the test)
 - tests: is it possible that shutil.chown() exists whereas pwd or grp module is missing? if yes, you should split the test into two parts. it looks like pwd and grp are avaiable on Unix.
 - os.chown(path, -1, -1) is accepted, why not accepting shutil.chown(path) (shutil.chown(path, None, None))?
 - style: i don't like _user name, i suggest uid and gid (or user_id and group_id) instead of _user and _group... or you can reuse user/group variables
 - style: "else: \n if not isinstance(user, int):" can be written "elif not isinstance(user, int):" to avoid an useless level of indentation
 - tests: you may only call os.getuid() and os.getgid() once
History
Date User Action Args
2011-06-19 22:57:15vstinnersetrecipients: + vstinner, gregory.p.smith, ncoghlan, eric.smith, giampaolo.rodola, eric.araujo, sandro.tosi, neologix, petri.lehtinen
2011-06-19 22:57:15vstinnersetmessageid: <1308524235.63.0.345095182484.issue12191@psf.upfronthosting.co.za>
2011-06-19 22:57:15vstinnerlinkissue12191 messages
2011-06-19 22:57:14vstinnercreate