-
Notifications
You must be signed in to change notification settings - Fork 362
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
Replace deprecated CURLINFO_CONTENT_LENGTH_DOWNLOAD #8659
Conversation
Yes, it's deprecated since curl 7.55 (https://curl.se/libcurl/c/CURLINFO_CONTENT_LENGTH_DOWNLOAD.html), and curl 7.55 was released in 2017 (https://curl.se/docs/releases.html), so I think it should be good. Maybe we should also set the minimum required curl version in the following line Line 143 in 79800bd
|
Yes, that minimum curl version would be 7.55.0 then. I'll work out how to do that and add to the PR. |
@seisman: Done as requested |
ff08555
to
c89157d
Compare
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.
Looks good to me.
@remkos Next time, you can just ping @GenericMappingTools/gmt-maintainers for code reviews. This team contains people who maintain/change the codes. See https://github.com/orgs/GenericMappingTools/teams/gmt-maintainers for the list of people. On the other hand, if you want to be pinged for PR reviews, I can also add you to the team. |
Thanks for the suggestions, @seisman. I have already added myself to @GenericMappingTools/gmt-contributors and will use that in future review requests. |
We should have only one group for asking reviews. The way we have it it's too confusing what-is-what-and-what-for. |
Description of proposed changes
Compiling on MacOS gave this warning:
So I have replaced
CURLINFO_CONTENT_LENGTH_DOWNLOAD
byCURLINFO_CONTENT_LENGTH_DOWNLOAD_T
and changed the type offilesize
fromdouble
tocurl_off_t
.The addresses #8653
Reminders