Skip to content

[curl] Update to 8.19.0 [wolfssl] Fixup#50125

Merged
vicroms merged 16 commits intomicrosoft:masterfrom
dg0yt:curl
Mar 13, 2026
Merged

[curl] Update to 8.19.0 [wolfssl] Fixup#50125
vicroms merged 16 commits intomicrosoft:masterfrom
dg0yt:curl

Conversation

@dg0yt
Copy link
Copy Markdown
Contributor

@dg0yt dg0yt commented Feb 23, 2026

No description provided.

dg0yt referenced this pull request in curl/curl Feb 23, 2026
To fix potential `-Wdisabled-macro-expansion` warnings when using these
macros within other macros. Fixing for example:
```
lib/doh.c:328:3: error: disabled expansion of recursive macro [clang-diagnostic-disabled-macro-expansion,-warnings-as-errors]
  328 |   ERROR_CHECK_SETOPT(CURLOPT_URL, url);
      |   ^
lib/doh.c:271:14: note: expanded from macro 'ERROR_CHECK_SETOPT'
  271 |     result = curl_easy_setopt((CURL *)doh, x, y);       \
      |              ^
include/curl/curl.h:3332:44: note: expanded from macro 'curl_easy_setopt'
 3332 | #define curl_easy_setopt(handle,opt,param) curl_easy_setopt(handle,opt,param)
      |                                            ^
[...]
```

Also update comments on why curl continues to disable
`-Wdisabled-macro-expansion` and `-Wused-but-marked-unused` warnings.

Follow-up to 92f215f #18477

Closes #20597
@dg0yt dg0yt changed the title [curl] 8.19.0 rc2 [curl] Update to 8.19.0 Mar 11, 2026
HEAD_REF master
PATCHES
)
have-limits-h.diff
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dg0yt dg0yt changed the title [curl] Update to 8.19.0 [curl] Update to 8.19.0 [wolfssl] Fixup Mar 11, 2026
@dg0yt
Copy link
Copy Markdown
Contributor Author

dg0yt commented Mar 11, 2026

arm64-osx is a pkgconf bug, exposed by CMake.
pkgconf/pkgconf#462

dg0yt added 2 commits March 12, 2026 01:27
This reverts commit 361b46b.
@dg0yt dg0yt marked this pull request as ready for review March 12, 2026 11:42
@vicroms vicroms merged commit 48cfe1e into microsoft:master Mar 13, 2026
16 checks passed
@dg0yt dg0yt deleted the curl branch March 15, 2026 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants