Issue 20868: Lib/test/test_socket.py: skip testGetServBy if /etc/services is not found (original) (raw)

Created on 2014-03-07 22:57 by Steap, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
skip_testGetServBy.patch Steap,2014-03-07 22:57 review
Messages (6)
msg212910 - (view) Author: Steap (Steap) Date: 2014-03-07 22:57
In Lib/test/test_socket.py, testGetServBy calls socket.getservbyname(), which needs /etc/services (see "man getservbyname"). If this file is not found, the test fails instead of being skipped. The attached patch was written against the latest revision of the Mercurial repository. It might be worth applying it for every currently supported version of Python.
msg212912 - (view) Author: Ned Deily (ned.deily) * (Python committer) Date: 2014-03-07 23:20
Unfortunately, how getservbyname() and other similar network interface functions get their data is platform-dependent. /etc/services is a traditional file location but many modern systems use a database or shared database (e.g. NIS) and even allow the system administrator to dynamically change the source of the data. So, checking for /etc/services would cause the test to be skipped needlessly. I would think that most systems would not be very usable without a working getservbyname(). Under what circumstances is this causing a problem for you?
msg213016 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2014-03-10 08:46
I don't understand why /etc/services would miss. What is the current behaviour? What is the error message?
msg213635 - (view) Author: Steap (Steap) Date: 2014-03-15 04:31
I did not know that /etc/services is not always used :/ The main issue is that when you run the test suite in a chroot (to do "pure" builds) or in a network sandbox (for other functions). I get the following errors: ====================================================================== ERROR: testGetServBy (test.test_socket.GeneralModuleTests) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/nix-build-python-3.3.4.drv-0/Python-3.Lib/test/test_socket.py", line 840, in testGetServBy raise socket.error OSError ====================================================================== ERROR: testGetaddrinfo (test.test_socket.GeneralModuleTests) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/nix-build-python-3.3.4.drv-0/Python-3.Lib/test/test_socket.py", line 1126, in testGetaddrinfo socket.getaddrinfo(HOST, "http") socket.gaierror: [Errno -8] Servname not supported for ai_socktype ====================================================================== ERROR: test_idna (test.test_socket.GeneralModuleTests) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/nix-build-python-3.3.4.drv-0/Python-3.Lib/test/test_socket.py", line 1186, in test_idna socket.gethostbyname('\u0438\u0441\u043f\u044b\u0442\u0430\u043d\u0438\u0435.python.org') socket.gaierror: [Errno -2] Name or service not known
msg406598 - (view) Author: Irit Katriel (iritkatriel) * (Python committer) Date: 2021-11-19 17:05
> when you run the test suite in a chroot (to do "pure" builds) or in a network sandbox (for other functions) Do we support that?
msg406599 - (view) Author: Christian Heimes (christian.heimes) * (Python committer) Date: 2021-11-19 17:10
Nope, we don't have to support this edge case.
History
Date User Action Args
2022-04-11 14:57:59 admin set github: 65067
2021-11-19 17:10:36 christian.heimes set status: pending -> closednosy: + christian.heimesmessages: + resolution: rejectedstage: resolved
2021-11-19 17:05:41 iritkatriel set status: open -> pendingnosy: + iritkatrielmessages: +
2014-03-15 04:31:16 Steap set messages: +
2014-03-10 08:46:13 vstinner set messages: +
2014-03-10 08:45:17 vstinner set nosy: + vstinner
2014-03-07 23:20:42 ned.deily set nosy: + ned.deilymessages: +
2014-03-07 22:57:22 Steap create