]> granicus.if.org Git - python/commitdiff
Merged revisions 88352 via svnmerge from
authorGregory P. Smith <greg@mad-scientist.com>
Sat, 5 Feb 2011 21:51:27 +0000 (21:51 +0000)
committerGregory P. Smith <greg@mad-scientist.com>
Sat, 5 Feb 2011 21:51:27 +0000 (21:51 +0000)
svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r88352 | gregory.p.smith | 2011-02-05 13:47:25 -0800 (Sat, 05 Feb 2011) | 3 lines

  issue7678 - Properly document how to replace a shell pipeline so that SIGPIPE
  happens when the end exits before the beginning.
........

Doc/library/subprocess.rst

index c49020346832bfc0247d7bd064073444ad2e56d4..e333309c16a1f6cf04f6c444f491d5df690a1179 100644 (file)
@@ -465,8 +465,11 @@ Replacing shell pipeline
    ==>
    p1 = Popen(["dmesg"], stdout=PIPE)
    p2 = Popen(["grep", "hda"], stdin=p1.stdout, stdout=PIPE)
+   p1.stdout.close()  # Allow p1 to receive a SIGPIPE if p2 exits.
    output = p2.communicate()[0]
 
+The p1.stdout.close() call after starting the p2 is important in order for p1
+to receive a SIGPIPE if p2 exits before p1.
 
 Replacing :func:`os.system`
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^