◐ Shell
reader mode source ↗
Skip to content

bpo-31904: Add subprocess module support for VxWorks RTOS#12157

Closed
pxinwr wants to merge 7 commits into
python:masterfrom
Wind-River:fix-issue-31904-subprocess
Closed

bpo-31904: Add subprocess module support for VxWorks RTOS#12157
pxinwr wants to merge 7 commits into
python:masterfrom
Wind-River:fix-issue-31904-subprocess

Conversation

@pxinwr

@pxinwr pxinwr commented Mar 4, 2019

Copy link
Copy Markdown
Contributor

This is the successive PR after #11968. This PR enables subprocess module support for VxWorks RTOS. VxWorks doesn't support fork()/exec(). Instead, we use rtpSpawn() to spawn new processes. So a new extension module named _vxwapi is added to offer the method rtp_spawn, which will be used in subprocess module to create new processes.
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

@bedevere-bot

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@pxinwr

pxinwr commented Mar 5, 2019

Copy link
Copy Markdown
Contributor Author

I have made the requested changes; please review again

@bedevere-bot

Copy link
Copy Markdown

Thanks for making the requested changes!

@gpshead: please review the changes made to this pull request.

@pxinwr

pxinwr commented Apr 3, 2020

Copy link
Copy Markdown
Contributor Author

Close it. No response for long time and also need some rebase actions.

@pxinwr pxinwr closed this Apr 3, 2020
@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants