This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eric.smith
Recipients berker.peksag, eric.smith, lars.gustaebel, mvo, r.david.murray, serhiy.storchaka
Date 2015-04-14.17:35:28
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1429032929.43.0.653312239514.issue23193@psf.upfronthosting.co.za>
In-reply-to
Content
Thanks for your review, Berker. I've updated the code with most of your suggestions, although some of them were mooted by some restructuring I did.

A couple of questions/issues:
- I'm not sure where we stand on keyword-only arguments. I certainly agree that I'd never specify numeric_only unless I named it. However, I don't see a lot of that style in new or modified APIs. I'll ask over on python-dev and see what they say.

- test_extract_without_numeric_owner only works if the user named 'root' has uid = 0 (same for gid). This is a different test than what test_os.root_in_posix is doing. I think it's best to leave it as-is, although I've added a comment and reduced the scope of the skip to just this one test.

- I reformatted the tests to stay within 80 characters, and I think it also made them more legible.

- I'm not sure what you mean by your last point. I believe I use True as it is elsewhere in that module and its documentation. And None doesn't make any sense to me as a parameter value for this.
History
Date User Action Args
2015-04-14 17:35:29eric.smithsetrecipients: + eric.smith, lars.gustaebel, mvo, r.david.murray, berker.peksag, serhiy.storchaka
2015-04-14 17:35:29eric.smithsetmessageid: <1429032929.43.0.653312239514.issue23193@psf.upfronthosting.co.za>
2015-04-14 17:35:29eric.smithlinkissue23193 messages
2015-04-14 17:35:29eric.smithcreate