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
bpo-31904: fix fifo test cases for VxWorks RTOS #20254
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks OK!
Thanks for your review. |
8ca70ef
to
de57a99
Compare
Note: The commit 6a273fd skipped FIFO tests in test_pathlib and test_shutil. I'm fine with these skips. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. The updated PR is simple enough and is not too intrusive.
If there are more cases tomorrow, it might be worth it to add an helper function to support.os_helper, but I don't think that it's the case now (it's only used in exactly two functions).
In VxWork RTOS, FIFO must be created under directory "/fifos/". So tuned related test cases accordingly.
https://bugs.python.org/issue31904