Skip to content

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Oct 24, 2025

Tested OK with full non-unity CI run.

@vszakats vszakats marked this pull request as draft October 24, 2025 16:47
@vszakats vszakats marked this pull request as ready for review October 24, 2025 17:19
@github-actions github-actions bot added the CI Continuous Integration label Oct 24, 2025
…lds'

Not tested in default CI.

macOS / CM clang OpenSSL gsasl rtmp AppleIDN SecTrust +examples:
macOS / CM llvm@18 OpenSSL gsasl rtmp AppleIDN SecTrust +examples:
```
lib/vquic/curl_ngtcp2.c:530:5: error: code will never be executed [-Werror,-Wunreachable-code]
  530 |     const ngtcp2_transport_params *rp;
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
```
Ref: https://github.com/curl/curl/actions/runs/18787154442/job/53608230871?pr=19225#step:11:183
@vszakats vszakats changed the title lib: drop unused crypto headers lib: delete unused crypto header includes Oct 24, 2025
@vszakats vszakats closed this in 3fc7277 Oct 24, 2025
vszakats added a commit that referenced this pull request Oct 24, 2025
Not tested in default CI.

macOS / CM clang OpenSSL gsasl rtmp AppleIDN SecTrust +examples,
macOS / CM llvm@18 OpenSSL gsasl rtmp AppleIDN SecTrust +examples:
```
lib/vquic/curl_ngtcp2.c:530:5: error: code will never be executed [-Werror,-Wunreachable-code]
  530 |     const ngtcp2_transport_params *rp;
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
```
Ref: https://github.com/curl/curl/actions/runs/18787154442/job/53608230871?pr=19225#step:11:183

Confirmed fixed via #19225
Closes #19226
@vszakats vszakats deleted the unused-include branch October 24, 2025 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration tidy-up

Development

Successfully merging this pull request may close these issues.

1 participant