Skip to content

bpo-41586: Add pipesize parameter to subprocess. Add F_GETPIPE_SZ and F_SETPIPE_SZ to fcntl. #21921

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.

Sign up for GitHub

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

Merged
gpshead merged 17 commits into from
Oct 19, 2020
Merged

bpo-41586: Add pipesize parameter to subprocess. Add F_GETPIPE_SZ and F_SETPIPE_SZ to fcntl. #21921

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
1e51d96
Add F_SETPIPE_SZ and F_GETPIPE_SZ to fcntl module
rhpvorderman Aug 18, 2020
a218c39
Add pipesize parameter for Popen class
rhpvorderman Aug 18, 2020
02b1d43
Add test for F_SETPIPE_SZ and F_GETPIPE_SZ
rhpvorderman Aug 18, 2020
af05503
Ensure cross-platform compatibility for pipesize in subprocess.Popen
rhpvorderman Aug 18, 2020
493d6c3
Add linux test for pipesize in subprocess
rhpvorderman Aug 18, 2020
aad24c6
Enable subprocess pipesize test for all platforms
rhpvorderman Aug 19, 2020
acf9ae4
Do not set pipesizes for Popen on Windows, as this can not be tested …
rhpvorderman Aug 19, 2020
6815aaa
Add F_GETPIPE_SZ, F_SETPIPE_SZ and subprocess.Popen pipesize argument…
rhpvorderman Aug 19, 2020
6bfd6a5
Add Ruben Vorderman to ACKS for work on the pipesize parameter in sub…
rhpvorderman Aug 19, 2020
cf97235
Add blurb for bpo 41586
rhpvorderman Aug 19, 2020
389e4e9
Simplify test fore F_GETPIPE_SZ and F_SETPIPE_SZ so no file cleanup i…
rhpvorderman Aug 19, 2020
e601174
Test F_SETPIPE_SZ and F_GETPIPE_SZ on all supported platforms
rhpvorderman Aug 19, 2020
24f061b
Intentionally exclude fcntl from subprocess.__all__ test.
rhpvorderman Aug 19, 2020
58925f3
Properly format the pipesize argument in the subprocess docs.
rhpvorderman Aug 19, 2020
15b759e
Determine default dynamically
rhpvorderman Aug 24, 2020
d4d0886
Add test for pipesize default
rhpvorderman Aug 24, 2020
c129ca8
Add a note to the documentation about only Linux supporting the pipes…
rhpvorderman Aug 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Properly format the pipesize argument in the subprocess docs.
  • Loading branch information
@rhpvorderman
rhpvorderman committed Aug 19, 2020
commit 58925f3bdeaa904b64e8e5257c594c1268c9de47
5 changes: 3 additions & 2 deletions Doc/library/subprocess.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -625,8 +625,9 @@ functions.
* :data:`CREATE_DEFAULT_ERROR_MODE`
* :data:`CREATE_BREAKAWAY_FROM_JOB`

``pipesize`` can be used to change the size of the pipe when
``subprocess.PIPE`` is used for ``stdin``,``stdout`` or ``stderr`` on Linux.
*pipesize* can be used to change the size of the pipe when
:data:`PIPE` is used for *stdin*, *stdout* or *stderr*. The size of the pipe
is only changed on platforms that support this.

@gpshead gpshead Aug 21, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mention something like (Linux at the time of this writing) at the end here.

that's enough of a documentation carrot to cause someone who wants this on Windows, macOS, or some BSD to pop up and contribute support on that platform in the future.


.. versionadded:: 3.10
The ``pipesize`` parameter was added.
Expand Down