Skip to content

curl.h: switch CURL_HTTP_VERSION* enums to long constants#16580

Closed
vszakats wants to merge 3 commits intocurl:masterfrom
vszakats:easy-long-2
Closed

curl.h: switch CURL_HTTP_VERSION* enums to long constants#16580
vszakats wants to merge 3 commits intocurl:masterfrom
vszakats:easy-long-2

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Mar 5, 2025

It fixes tests 1539, and 2402, 2404 (for non-Secure Transport), on macOS
with the gcc compiler.

Also unignore these tests in GHA/macos for non-secure transport.

Ref: c349bd6 #14097 (issue 15.)
Ref: 7b0240c #16539
Ref: 2ec0037 #16482

@vszakats vszakats added HTTP tests appleOS specific to an Apple operating system labels Mar 5, 2025
@github-actions github-actions bot added the CI Continuous Integration label Mar 5, 2025
@vszakats vszakats changed the title test1539, 2402, 2404: switch enums to long constants test1539, 2402, 2404: switch CURL_HTTP_VERSION* enums to long constants Mar 5, 2025
@vszakats vszakats changed the title test1539, 2402, 2404: switch CURL_HTTP_VERSION* enums to long constants curl.h: switch CURL_HTTP_VERSION* enums to long constants Mar 5, 2025
@vszakats vszakats closed this in fd54bed Mar 6, 2025
@vszakats vszakats deleted the easy-long-2 branch March 6, 2025 00:02
@jcourreges
Copy link

jcourreges commented Apr 4, 2025

[EDIT] suppressed error due to an unrelated commit, sorry for the noise.

@vszakats
Copy link
Member Author

vszakats commented Apr 4, 2025

[EDIT] suppressed error due to an unrelated commit, for the noise.

That isn't related to this commit, but 2ec0037 #16482, where it was reported earlier, and I linked the CMake patch that fixed it downstream: #16482 (comment)

I think the answer is CMake was using a non-public type here (but I may be confusing this with another case and no time to archeologe into this now.)

pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
It fixes tests 1539, and 2402, 2404 (for non-Secure Transport), on macOS
with the gcc compiler.

Also unignore these tests in GHA/macos for non-secure transport.

Ref: c349bd6 curl#14097 (issue 15.)
Ref: 7b0240c curl#16539
Ref: 2ec0037 curl#16482

Closes curl#16580
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

appleOS specific to an Apple operating system CI Continuous Integration HTTP libcurl API tests

Development

Successfully merging this pull request may close these issues.

3 participants