Skip to content

config-win32: set HAVE_SNPRINTF for mingw-w64#12325

Closed
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:win-snprintf-mingw
Closed

config-win32: set HAVE_SNPRINTF for mingw-w64#12325
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:win-snprintf-mingw

Conversation

@vszakats

@vszakats vszakats commented Nov 15, 2023

Copy link
Copy Markdown
Member

It's available in all mingw-w64 releases. We already pre-fill this
detection in CMake.

Closes #12325

@vszakats vszakats added the build label Nov 15, 2023
@github-actions github-actions Bot added the Windows Windows-specific label Nov 15, 2023
vszakats added a commit to curl/curl-for-win that referenced this pull request Nov 15, 2023
- enable symbol hiding manually to match cmake/autotools.

- enable variadic macros. These macros became required in curl 8.3.0
  with curl/curl#11421.
  These macros will no longer be needed with curl 8.5.0
  after curl/curl#12167.

- set `HAVE_SNPRINTF`. curl's `config-win32.h` missed to set it,
  causing a minor difference in the generated binaries compared to
  cmake/autotools builds.
  Ref: curl/curl#12325

With these changes GNU Make builds are in sync again with
cmake/autotools.
@vszakats vszakats closed this in 6608f34 Nov 15, 2023
@vszakats vszakats deleted the win-snprintf-mingw branch November 15, 2023 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Windows Windows-specific

Development

Successfully merging this pull request may close these issues.

1 participant