Message99919
| Author |
Alexander.Belopolsky |
| Recipients |
Alexander.Belopolsky, l0nwlf, loewis, michael.foord, orsenthil, r.david.murray, ronaldoussoren |
| Date |
2010年02月23日.14:42:06 |
| SpamBayes Score |
2.68785e-13 |
| Marked as misclassified |
No |
| Message-id |
<d38f5331002230642q4cdab764kffeb3fa96c2b62bc@mail.gmail.com> |
| In-reply-to |
<1266932795.31.0.207724978264.issue7900@psf.upfronthosting.co.za> |
| Content |
On Tue, Feb 23, 2010 at 8:46 AM, Ronald Oussoren <report@bugs.python.org> wrote:
..
> What I did notice though is that the tests don't actually test the issue at hand:
> having more than 16 secondairy groups on OSX.
This is easy to fix: just add another entry in groups
+ def test_setgroups(self):
+ for groups in [[0], range(16)]:
I did not do it for two reasons:
1. Laziness. :-) It was easier for me to just replace "16" above with
other values in my tests than to introduce platform checks. I believe
16 groups is supported on all relevant platforms.
2. I am not sure python should support non-posix behavior and allow
more than os.sysconf('SC_NGROUPS_MAX') groups to be returned from
os.getgroups(). IMO, users who need more than 16 groups on OSX should
bug Apple to fix sysconf(_SC_NGROUPS_MAX). |
|