8000 gh-82616: Add process_group support to subprocess.Popen by gpshead · Pull Request #23930 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-82616: Add process_group support to subprocess.Popen #23930

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
merged 18 commits into from
May 5, 2022
Merged

gh-82616: Add process_group support to subprocess.Popen #23930

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
1454088
Add setpgid support to subprocess.POpen
gpshead Dec 25, 2020
6aad6e3
Fix compilation error.
gpshead Dec 25, 2020
5f5d800
Fix the fork_exec() calls. tests pass.
gpshead Dec 25, 2020
010ae9a
Fix multiprocessing's _posixsubprocess.fork_exec() call.
gpshead Dec 25, 2020
8a770ab
Fix test_capi.
gpshead Dec 25, 2020
645256e
Merge branch 'main' into subprocess-issue38435
gpshead May 5, 2022
b58758b
Add missing \ in Doc/library/subprocess.rst
gpshead May 5, 2022
acba394
better ReST in Doc/library/subprocess.rst
gpshead May 5, 2022
cef3bbe
improve formatting in Doc/library/subprocess.rst
gpshead May 5, 2022
e594a01
improve formatting in Doc/library/subprocess.rst
gpshead May 5, 2022
ab9f0a5
better formatting
gpshead May 5, 2022
3b3a5d0
versionadded 3.11
gpshead May 5, 2022
20a1363
Rename from setpgid to process_group
gpshead May 5, 2022
3df5b30
test_capi: update its internal call
gpshead May 5, 2022
450e87d
Merge branch 'main' into subprocess-issue38435
gpshead May 5, 2022
44cc293
blindly attempting to appease an ASAN bug.
gpshead May 5, 2022
9d48704
Merge branch 'subprocess-issue38435' of github.com:gpshead/cpython in…
gpshead May 5, 2022
bb9461b
Disable the tests exposing ASAN flakiness.
gpshead May 5, 2022
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
Fix test_capi.
  • Loading branch information
@gpshead
gpshead committed Dec 25, 2020
commit 8a770ab6ff35977dc0f92be8fa5e166aaf91891e
6 changes: 3 additions & 3 deletions Lib/test/test_capi.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,15 +108,15 @@ class Z(object):
def __len__(self):
return 1
self.assertRaises(TypeError, _posixsubprocess.fork_exec,
1,Z(),3,(1, 2),5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21)
1,Z(),3,(1, 2),5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22)
# Issue #15736: overflow in _PySequence_BytesToCharpArray()
class Z(object):
def __len__(self):
return sys.maxsize
def __getitem__(self, i):
return b'x'
self.assertRaises(MemoryError, _posixsubprocess.fork_exec,
1,Z(),3,(1, 2),5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21)
1,Z(),3,(1, 2),5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22)

@unittest.skipUnless(_posixsubprocess, '_posixsubprocess required for this test.')
def test_subprocess_fork_exec(self):
Expand All @@ -126,7 +126,7 @@ def __len__(self):

# Issue #15738: crash in subprocess_fork_exec()
self.assertRaises(TypeError, _posixsubprocess.fork_exec,
Z(),[b'1'],3,(1, 2),5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21)
Z(),[b'1'],3,(1, 2),5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22)

@unittest.skipIf(MISSING_C_DOCSTRINGS,
"Signature information for builtins requires docstrings")
Expand Down
0