Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmake: detect HAVE_NETINET_IN6_H, HAVE_CLOSESOCKET_CAMEL, HAVE_PROTO_BSDSOCKET_H #15172

Closed
wants to merge 3 commits into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Oct 7, 2024

To sync with ./configure.

@dfandrich
Copy link
Contributor

Analysis of PR #15172 at c2364c58:

Test 504 failed, which has NOT been flaky recently, so there could be a real issue in the PR.

Generated by Testclutch

do not add socket.h

committed to configure in 37eba37
to detect socket and closesocket, but then this header is nowhere
used from the code itself, suggesting this was unnecessary.

Let's not add this to cmake. It probably could be deleted from
configure.

limit HAVE_CLOSESOCKET_CAMEL to AmigaOS

limit HAVE_PROTO_BSDSOCKET_H to AmigaOS
Needed by HPE NonStop NSE and NSX systems

Follow-up to 76ebd54 curl#2155
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants