-
-
Notifications
You must be signed in to change notification settings - Fork 29.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mark test_devpoll to be meaningful only for Solaris #65515
Comments
[ 96/389/2] test_devpoll Took me a while until I read documentation: Even test itself, test_devpoll.py, doesn't mention it is only for Solaris. |
New changeset cc2345e6e9ff by Jesus Cea in branch '3.4': New changeset 825c67196aac by Jesus Cea in branch 'default': |
Thanks for the head-up!!. Can you compile new version and try it out?. |
There's no pressing reason to revisit this, but...I think it would be better to say "works only on systems that support devpoll (ie: Solaris and derivatives)" Also, as long as you are changing the try/except, you could use skipUnless(hasattr(select, 'devpoll'), ...) instead of an if. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: