diff options
author | Michał Górny <mgorny@gentoo.org> | 2021-07-28 10:56:04 +0200 |
---|---|---|
committer | Michał Górny <mgorny@gentoo.org> | 2021-07-28 11:07:59 +0200 |
commit | c8005f10a737943a4e7a7d06ffd78fb71b2d09b4 (patch) | |
tree | 0e21968a73ec3900860dba1b58e2574d622e508a /dev-python/requests | |
parent | dev-python/fsspec: Remove old (diff) | |
download | gentoo-c8005f10a737943a4e7a7d06ffd78fb71b2d09b4.tar.gz gentoo-c8005f10a737943a4e7a7d06ffd78fb71b2d09b4.tar.bz2 gentoo-c8005f10a737943a4e7a7d06ffd78fb71b2d09b4.zip |
dev-python/requests: Fix test failures due to listen(0)
Fix test failures on platforms (alpha, riscv) that literally interpret
listen(0) as 'zero connections allowed'.
Closes: https://bugs.gentoo.org/803515
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Diffstat (limited to 'dev-python/requests')
-rw-r--r-- | dev-python/requests/files/requests-2.26.0-test.patch | 30 | ||||
-rw-r--r-- | dev-python/requests/requests-2.26.0.ebuild | 4 |
2 files changed, 34 insertions, 0 deletions
diff --git a/dev-python/requests/files/requests-2.26.0-test.patch b/dev-python/requests/files/requests-2.26.0-test.patch new file mode 100644 index 000000000000..6bbf78871713 --- /dev/null +++ b/dev-python/requests/files/requests-2.26.0-test.patch @@ -0,0 +1,30 @@ +From 6552e5374c335d62ba960a74b5b978897c823376 Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Micha=C5=82=20G=C3=B3rny?= <mgorny@gentoo.org> +Date: Wed, 28 Jul 2021 10:33:22 +0200 +Subject: [PATCH] Fix creating non-listening sockets in tests on some platforms + +Fix the listen() invocation for the test server not to pass a backlog +value of zero. The value of zero means no backlog which effectively +means that the socket can not accept any connections. This does not +matter for the majority of platforms since the value is only advisory +and the platform tends to go with a bigger backlog anyway. However, +a few platforms (e.g. alpha or riscv Linux) do take the value literally, +and therefore the tests fail since they are unable to connect to +the server. Let Python use a 'default reasonable value' instead. +--- + tests/testserver/server.py | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/tests/testserver/server.py b/tests/testserver/server.py +index 132221f7c4..92dcb6cb1a 100644 +--- a/tests/testserver/server.py ++++ b/tests/testserver/server.py +@@ -78,7 +78,7 @@ def run(self): + def _create_socket_and_bind(self): + sock = socket.socket() + sock.bind((self.host, self.port)) +- sock.listen(0) ++ sock.listen() + return sock + + def _close_server_sock_ignore_errors(self): diff --git a/dev-python/requests/requests-2.26.0.ebuild b/dev-python/requests/requests-2.26.0.ebuild index 24926975d014..d7001ec67dfb 100644 --- a/dev-python/requests/requests-2.26.0.ebuild +++ b/dev-python/requests/requests-2.26.0.ebuild @@ -36,6 +36,10 @@ BDEPEND=" distutils_enable_tests pytest +PATCHES=( + "${FILESDIR}"/${P}-test.patch +) + python_test() { local deselect=( # Internet |