Skip to content

bpo-31904: Fix test_os failures for VxWorks RTOS #12319

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

Closed
wants to merge 2 commits into from

Conversation

yuesun1
Copy link

@yuesun1 yuesun1 commented Mar 14, 2019

This is the successive PR after #11968 and #12118. The resource.setrlimit() can't set NOFILE to 1 on VxWorks, we can only track the highest fd value in use, and only permit lowering NOFILE up to that value, so in testcase 'test_urandom_failure' I have to keep opening '/dev/null' until reporting errno EMFILE to make sure the fd table is full already and we can't open any more file.
And the VxWorks doesn't support execv like functions, so I skipped them.
More and full support on modules for VxWorks will continuously be added by the coming PRs.
VxWorks is a product developed and owned by Wind River. For VxWorks introduction or more details, go to https://www.windriver.com/products/vxworks/
Wind River will have a dedicated engineering team to contribute to the support as maintainers.
We already have a working buildbot worker internally, but has not bound to master. We will check the process for the buildbot, then add it.

https://bugs.python.org/issue31904

@pxinwr
Copy link
Contributor

pxinwr commented Apr 2, 2019

@rhettinger @mark.dickinson Could you help on reviewing this? Thanks in advance.

@vstinner
Copy link
Member

I would prefer to see #12118 merged yet and/or combine the two PR.

@pxinwr
Copy link
Contributor

pxinwr commented May 17, 2019

@yuesun1 Please close this PR. I've combined this PR with #12118 as @vstinner required.

@vstinner vstinner closed this May 17, 2019
@vstinner
Copy link
Member

I closed it ;-)

@pxinwr pxinwr deleted the fix-issue-31904-test-os branch April 9, 2020 06:15
@kuhlenough kuhlenough mannequin mentioned this pull request Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants