[issue20868] Lib/test/test_socket.py: skip testGetServBy if /etc/services is not found

Christian Heimes added the comment: Nope, we don't have to support this edge case. -- nosy: +christian.heimes resolution: -> rejected stage: -> resolved status: pending -> closed ___ Python tracker ___

[issue20868] Lib/test/test_socket.py: skip testGetServBy if /etc/services is not found

Irit Katriel added the comment: > 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? -- nosy: +iritkatriel status: open -> pending ___ Python tracker

[issue20868] Lib/test/test_socket.py: skip testGetServBy if /etc/services is not found

Steap added the comment: 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:

[issue20868] Lib/test/test_socket.py: skip testGetServBy if /etc/services is not found

STINNER Victor added the comment: I don't understand why /etc/services would miss. What is the current behaviour? What is the error message? -- ___ Python tracker ___ __

[issue20868] Lib/test/test_socket.py: skip testGetServBy if /etc/services is not found

Changes by STINNER Victor : -- nosy: +haypo ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue20868] Lib/test/test_socket.py: skip testGetServBy if /etc/services is not found

Ned Deily added the comment: 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 admini

[issue20868] Lib/test/test_socket.py: skip testGetServBy if /etc/services is not found

New submission from Steap: 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 Mercuria