Repository navigation
Popen() object stdout attribute reassignment behaviour #47937
Description
| BPO | 3687 |
|---|---|
| Nosy | @birkenfeld, @terryjreedy, @giampaolo, @maaz92, @iritkatriel |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = None
closed_at = None
created_at = <Date 2008-08-26.15:57:45.065>
labels = ['3.11', 'type-bug', '3.9', '3.10', 'docs']
title = 'Popen() object stdout attribute reassignment behaviour'
updated_at = <Date 2021-12-05.15:57:52.556>
user = 'https://bugs.python.org/vincentlegoll'bugs.python.org fields:
activity = <Date 2021-12-05.15:57:52.556>
actor = 'iritkatriel'
assignee = 'docs@python'
closed = False
closed_date = None
closer = None
components = ['Documentation']
creation = <Date 2008-08-26.15:57:45.065>
creator = 'vincent.legoll'
dependencies = []
files = []
hgrepos = []
issue_num = 3687
keywords = []
message_count = 5.0
messages = ['71983', '107970', '107976', '281545', '407713']
nosy_count = 7.0
nosy_names = ['georg.brandl', 'terry.reedy', 'giampaolo.rodola', 'vlegoll', 'vincent.legoll', 'maaz92', 'iritkatriel']
pr_nums = []
priority = 'normal'
resolution = None
stage = 'needs patch'
status = 'open'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue3687'
versions = ['Python 3.9', 'Python 3.10', 'Python 3.11']
Activity
vincentlegoll commented on Aug 26, 2008
The subprocess.Popen() object documentation should indicate
that the stdout attribute should not be modified after
object construction. Because that won't work.
Or the attribute may be rendered read-only
This blocks forever
terryjreedy commented on Jun 17, 2010
While it could be argued that it is 'obvious' that changing the stdin, stdout, stderr, and pid of a processes cannot work, I agree that a small addition would be good. In 17.1.2. Popen Objects, after "The following attributes are also available", insert " (do not try to change them)" before ':'.
Actually, it seems to me that the attributes should actually be read-only (in 3.2) if possible (via custom Popen.__setattr__), in which case the only doc change needed (for 3.2) would be insertion of 'read-only' before 'attributes'.
Georg, if you know the id of the subprocess maintainer (if there is one now), could you add him to the nosy list for an opinion or comment?
vlegoll commented on Jun 17, 2010
On Thu, Jun 17, 2010 at 2:34 AM, Terry J. Reedy <report@bugs.python.org> wrote:
What is obvious for someone maybe is not for others, if I tried to modify it,
that was on the (false) assumption that it will do what I wanted (whatever
that is)...
Adding more precision to documentation or making the interface fool proof
will make life of dumb devs like me easier by not having to guess or try.
Thanks for taking care of this.
maaz92 commented on Nov 23, 2016
Hi, I want to update the doc for this. This will be my first attempt to update the documentation.
iritkatriel commented on Dec 5, 2021
The situation is the same in 3.11, both doc and implementation.
gh-47937: Note that Popen attributes are read-only (#93070)
pythongh-47937: Note that Popen attributes are read-only (pythonGH-93070
pythongh-47937: Note that Popen attributes are read-only (pythonGH-93070
pythongh-47937: Note that Popen attributes are read-only (pythonGH-93070
[3.11] gh-47937: Note that Popen attributes are read-only (GH-93070) (#…
[3.10] gh-47937: Note that Popen attributes are read-only (GH-93070) (#…
Merge branch 'main' into inlinecomp2