This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: cross-refs in the subprocess.Popen.std{in,out,err} warning box aren't linked
Type: enhancement Stage: resolved
Components: Documentation Versions: Python 3.2, Python 3.3, Python 3.4, Python 2.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: ezio.melotti Nosy List: cvrebert, docs@python, ezio.melotti, python-dev
Priority: normal Keywords: patch

Created on 2012-08-27 06:46 by cvrebert, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
link_communicate_warning.patch cvrebert, 2012-08-27 06:46 fix the markup so the x-refs are hyperlinked review
link_communicate_warning.patch cvrebert, 2012-08-27 06:54 per Ezio review
Messages (6)
msg169182 - (view) Author: Chris Rebert (cvrebert) * Date: 2012-08-27 06:46
The cross-references, particularly to Popen.communicate(), in the warning box in the subprocess docs about using Popen.stdout.read() etc. aren't hyperlinked. The attached patch fixes that.
msg169183 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2012-08-27 06:48
+ Use :meth:`communicate() <Popen.communicate>`

I believe you can simply use :meth:`~Popen.communicate` here.
msg169184 - (view) Author: Chris Rebert (cvrebert) * Date: 2012-08-27 06:52
So you can. Neat trick.
Patch changed accordingly.
msg169185 - (view) Author: Chris Rebert (cvrebert) * Date: 2012-08-27 06:54
Let's try that again.
msg169186 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2012-08-27 07:06
New changeset 92aa438a5c4b by Ezio Melotti in branch '2.7':
#15788: fix broken links in subprocess doc.  Patch by Chris Rebert.
http://hg.python.org/cpython/rev/92aa438a5c4b

New changeset 1676e423054a by Ezio Melotti in branch '3.2':
#15788: fix broken links in subprocess doc.  Patch by Chris Rebert.
http://hg.python.org/cpython/rev/1676e423054a

New changeset 64640a02b0ca by Ezio Melotti in branch 'default':
#15788: merge with 3.2.
http://hg.python.org/cpython/rev/64640a02b0ca
msg169187 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2012-08-27 07:07
Fixed, thanks for the patch!
History
Date User Action Args
2022-04-11 14:57:35adminsetgithub: 59992
2012-08-27 07:07:56ezio.melottisetstatus: open -> closed

assignee: docs@python -> ezio.melotti
versions: + Python 3.2
messages: + msg169187
type: enhancement
resolution: fixed
stage: resolved
2012-08-27 07:07:00python-devsetnosy: + python-dev
messages: + msg169186
2012-08-27 06:54:23cvrebertsetfiles: - link_communicate_warning.patch
2012-08-27 06:54:16cvrebertsetfiles: + link_communicate_warning.patch

messages: + msg169185
2012-08-27 06:52:42cvrebertsetfiles: + link_communicate_warning.patch

messages: + msg169184
2012-08-27 06:48:20ezio.melottisetnosy: + ezio.melotti
messages: + msg169183
2012-08-27 06:46:20cvrebertcreate