Skip to content

Commit

Permalink
Skip test_vsicurl_test_CPL_VSIL_CURL_USE_HEAD_NO() with buggy curl 8.9.1
Browse files Browse the repository at this point in the history
It turns out that curl 8.9.1 has a bug related to SIGPIPE handling which
caused SIGPIPE to no longer being ignored.
Cf curl/curl#14344 / curl/curl#14390
Fixes #10540
  • Loading branch information
rouault committed Aug 11, 2024
1 parent 8cb2522 commit 30a771e
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions autotest/gcore/vsicurl.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,13 +48,7 @@ def curl_version():
return actual_version


pytestmark = [
pytest.mark.require_curl(),
pytest.mark.skipif(
curl_version() >= [8, 9, 1],
reason="fail with SIGPIPE",
),
]
pytestmark = pytest.mark.require_curl()

###############################################################################
#
Expand Down Expand Up @@ -1058,6 +1052,11 @@ def test_vsicurl_GDAL_HTTP_HEADERS(server):
# Test CPL_VSIL_CURL_USE_HEAD=NO


# Cf fix of https://github.com/curl/curl/pull/14390
@pytest.mark.skipif(
curl_version() == [8, 9, 1],
reason="fail with SIGPIPE with curl 8.9.1",
)
def test_vsicurl_test_CPL_VSIL_CURL_USE_HEAD_NO(server):

gdal.VSICurlClearCache()
Expand Down

0 comments on commit 30a771e

Please sign in to comment.