Skip to content

build: drop HAVE_SYS_SOCKET_H and HAVE_SYS_TIME_H macros#17581

Closed
vszakats wants to merge 16 commits intocurl:masterfrom
vszakats:macros-drop
Closed

build: drop HAVE_SYS_SOCKET_H and HAVE_SYS_TIME_H macros#17581
vszakats wants to merge 16 commits intocurl:masterfrom
vszakats:macros-drop

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Jun 11, 2025

Both are available with well-known conditions, under non-Windows, and
curl/curl.h already uses them. sys/time.h is also necessary for
mingw-w64 for gettimeofday().

Follow-up to 56d5982 #17522

@vszakats vszakats marked this pull request as draft June 11, 2025 06:06
@github-actions github-actions bot added the tests label Jun 11, 2025
@vszakats vszakats marked this pull request as ready for review June 14, 2025 23:45
@vszakats vszakats closed this in 3186a20 Jun 15, 2025
@vszakats vszakats deleted the macros-drop branch June 15, 2025 16:45
denandz pushed a commit to denandz/curl that referenced this pull request Jun 21, 2025
Both are available with well-known conditions, under non-Windows, and
`curl/curl.h` already uses them. `sys/time.h` is also necessary for
mingw-w64 for `gettimeofday()`.

Follow-up to 56d5982 curl#17522

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

Labels

Development

Successfully merging this pull request may close these issues.

1 participant

Comments