[issue16170] remove Linux skip from test_subprocess's test_executable() test
Antoine Pitrou added the comment: Try prefixing doesnotexist with the same dirname as sys.executable (i.e. `os.path.join(os.path.dirname(sys.executable), doesnotexist)`). Python under Unix uses its own path to determine where the standard library is, and it does so by inspecting argv[0]. -- nosy: +pitrou ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Chris Jerdonek added the comment: Thanks for the explanation and suggestion, Antoine. I'm attaching a patch that incorporates the suggestion, but I haven't checked that it works yet (I would need to use snakebite). -- keywords: +patch Added file: http://bugs.python.org/file27507/issue-16170-1.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- stage: - patch review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Antoine Pitrou added the comment: Chris, you can also push a custom repo to a custom builder: http://docs.python.org/devguide/buildbots.html#custom-builders -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Antoine Pitrou added the comment: Actually, I've just checked that your patch works under Linux. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Chris Jerdonek added the comment: That's great, thanks for the pointer, Antoine. I will try that out later today. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Chris Jerdonek added the comment: Okay, even better. :) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Roundup Robot added the comment: New changeset 9451908da615 by Chris Jerdonek in branch '3.3': Issue #16170: Remove Linux skip from test_subprocess's test_executable test. http://hg.python.org/cpython/rev/9451908da615 New changeset 0ee03c9b098f by Chris Jerdonek in branch 'default': Issue #16170: Merge subprocess test unskip from 3.3. http://hg.python.org/cpython/rev/0ee03c9b098f -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Changes by Chris Jerdonek chris.jerdo...@gmail.com: -- resolution: - fixed stage: patch review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Chris Jerdonek added the comment: Python under Unix uses its own path to determine where the standard library is, and it does so by inspecting argv[0]. Actually, how is the above able to work if sys.argv[0] is set to -c when Python is invoked using the -c option (as it is in the test). http://docs.python.org/dev/library/sys.html#sys.argv -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Chris Jerdonek added the comment: Never mind on the above question. Those are different argv's. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
New submission from Chris Jerdonek: This issue is to make whatever changes are necessary to remove the skip from test_subprocess's test_executable() test (currently skipping machines that are neither Windows nor Mac): http://hg.python.org/cpython/file/ef90c5e482f4/Lib/test/test_subprocess.py#l203 The test currently fails on several Linux buildbots, but the test succeeds on Mac and Windows. Perhaps this points to a different, underlying issue that needs to be fixed. Either way, we should be able to test Popen()'s executable argument on Linux machines. This issue seems closely related to closed issue #7774. Below is more information from a buildbot about the failure (the AMD64 Ubuntu LTS buildbot): http://buildbot.python.org/all/builders/AMD64%20Ubuntu%20LTS%203.3/builds/38/steps/test/logs/stdio FAIL: test_executable (test.test_subprocess.ProcessTestCase) -- Traceback (most recent call last): File /opt/python/3.x.langa-ubuntu/build/Lib/test/test_subprocess.py, line 205, in test_executable self._assert_python([doesnotexist, -c], executable=sys.executable) File /opt/python/3.x.langa-ubuntu/build/Lib/test/test_subprocess.py, line 201, in _assert_python self.assertEqual(47, p.returncode) AssertionError: 47 != -6 == FAIL: test_executable (test.test_subprocess.ProcessTestCaseNoPoll) -- Traceback (most recent call last): File /opt/python/3.x.langa-ubuntu/build/Lib/test/test_subprocess.py, line 205, in test_executable self._assert_python([doesnotexist, -c], executable=sys.executable) File /opt/python/3.x.langa-ubuntu/build/Lib/test/test_subprocess.py, line 201, in _assert_python self.assertEqual(47, p.returncode) AssertionError: 47 != -6 test_executable (test.test_subprocess.ProcessTestCase) ... Could not find platform independent libraries prefix Could not find platform dependent libraries exec_prefix Consider setting $PYTHONHOME to prefix[:exec_prefix] Fatal Python error: Py_Initialize: Unable to get the locale encoding ImportError: No module named 'encodings' FAIL -- components: Library (Lib), Tests keywords: buildbot messages: 172432 nosy: chris.jerdonek priority: normal severity: normal status: open title: remove Linux skip from test_subprocess's test_executable() test type: behavior versions: Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue16170] remove Linux skip from test_subprocess's test_executable() test
Chris Jerdonek added the comment: For future reference, this issue resulted from the tests committed for issue 16115. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com