GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||
GeezerVison is showing github.com — bpo-29704: Fix asyncio.SubprocessStreamProtocol closing by sethmlarson · Pull Request #405 · python/cpython Conversation
Contributor
Was told by @1st1 to open this PR here from python/asyncio. See the original Pull Request for discussion and review: python/asyncio#485
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Member
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Contributor
Author
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Contributor
Author
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Contributor
Author
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
1st1
pushed a commit
to 1st1/cpython
that referenced
this pull request
This was referenced
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Contributor
Author
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
It's not so much about uvloop as it is about some unittests there that test both uvloop & vanilla asyncio. I plan to port some of those tests to CPython to increase coverage. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Anyways, thanks for the contribution! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@1st1 Would be a good idea to port them, I've got some tests for selectors2 that don't occur in Python's stdlib selectors that might be considered good as well for unique situations. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Please open a PR with those tests, selectors module is very important. For an extra benefit, you'll see your tests running on the full CPython buildbots matrix. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Will do, might have to experiment with different Python versions, PEP 475 really split the behavior for anything syscall-related in Python. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
What is the protocol for tests that behave differently (skip?) on different Python versions? Would they only be put in the version where they should be or is it normal to have tests that only run on Python 3.5+, for example. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Will do, might have to experiment with different Python versions, PEP 475
really split the behavior for anything syscall-related in Python.
I lost track of asyncio. PEP 475 was implemented in Python 3.5. Python 3.4
doesn't accept bugfixes anymore. Why do you want to test different Python
versions? Is asyncio on Python 3.3 or 3.4 still a thing?
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@Haypo Ah 3.4 is off bug-fix. Makes that a lot easier! :) Disregard my above comment then. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
We try to keep this table up to date:
https://docs.python.org/devguide/#status-of-python-branches
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@Haypo some rasberypi dev still on 3.4.2 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@Haypo some rasberypi dev still on 3.4.2
https://home-assistant.io this is still on 3.4.2
I had to ship backported cookies module with aiohttp.
You are on the CPython project. There is also a
http://github.com/python/asyncio project because there is no more
release on http://pypi.python.org/pypi/asyncio/ since 2 years. I
understand that asyncio doesn't evolve anymore on Python 3.4. Upgrade
or backport yourself, no?
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
* Use GH to signify Github handle of the user instead of "@". "@" usually doesn't give any information about what exactly the handle means since we don't know it is a Github handle from the commit message. Changing the pattern to be `GH:` similar to what we do for Pull Requests with (GH-). Fixes #270 * Handle removing GH:-style Automerge-Triggered-By ...when automerge label is removed by a core-dev. Co-authored-by: Abhilash Raj <[email protected]>
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.