-
Notifications
You must be signed in to change notification settings - Fork 7.5k
Bump curl from 8.14.0 to 8.15.0 #84231
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
Conversation
Workflow [PR], commit [d9f9115] Summary: ❌
|
799a1ff
to
46bb78b
Compare
contrib/curl-cmake/curl_config.h
Outdated
@@ -44,8 +44,8 @@ | |||
#define HAVE_FCNTL_H | |||
#define HAVE_NETDB_H | |||
#define HAVE_NETINET_IN_H | |||
#define HAVE_SELECT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be problematic. Where is it used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It became mandatory as per curl 8.15.0 (used by "lib/curlx/wait.c" which does
#ifndef HAVE_SELECT
#error "We cannot compile without select() support."
#endif
and uses timeval
in a wait function). It should be fine though, we include the same header from poco and c-ares already on all platforms.
Stateless tests (amd_binary, ParallelReplicas, s3 storage, parallel):
Integration tests (release, 3/4):
|
52750a6
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):
Bumped internal curl from 8.14.0 to 8.15.0