From: R David Murray <rdmurray@bitdance.com>
Date: Mon, 14 Mar 2011 02:37:18 +0000 (-0400)
Subject: #11490: EACCES can also mean command not found
X-Git-Tag: v2.7.2rc1~258
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cdd5fc90137023c9d2de336531b281f31740358f;p=python

#11490: EACCES can also mean command not found
---

diff --git a/Lib/test/test_subprocess.py b/Lib/test/test_subprocess.py
index 37dc245ebe..04eb9b14ff 100644
--- a/Lib/test/test_subprocess.py
+++ b/Lib/test/test_subprocess.py
@@ -573,7 +573,8 @@ class ProcessTestCase(BaseTestCase):
                 subprocess.Popen(['nonexisting_i_hope'],
                                  stdout=subprocess.PIPE,
                                  stderr=subprocess.PIPE)
-            if c.exception.errno != errno.ENOENT:  # ignore "no such file"
+            # ignore errors that indicate the command was not found
+            if c.exception.errno not in (errno.ENOENT, errno.EACCES):
                 raise c.exception
 
     def test_handles_closed_on_exception(self):
diff --git a/Misc/NEWS b/Misc/NEWS
index c3117cbcad..2b52138f2f 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -251,6 +251,9 @@ Build
 Tests
 -----
 
+- Issue #11490: test_subprocess:test_leaking_fds_on_error no longer gives a
+  false positive if the last directory in the path is inaccessible.
+
 - Issue #10822: Fix test_posix:test_getgroups failure under Solaris.  Patch
   by Ross Lagerwall.