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

os400: Sync libcurl API options #6084

Closed
wants to merge 1 commit into from
Closed

Conversation

jay
Copy link
Member

@jay jay commented Oct 15, 2020

This fixes the OS400 build and also an incorrect entry for
CURLINFO_APPCONNECT_TIME_T where it was treated as
CURLINFO_STARTTRANSFER_TIME_T.

Reported-by: Jon Rumsey

Fixes #6083
Closes #xxxx


/cc @jonrumsey

@jay jay added the build label Oct 15, 2020
Copy link
Contributor

@jonrumsey jonrumsey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, does curl_easy_setopt_ccsid() in packages/OS400/ccsidcurl.c need a case statement for CURLOPT_SSL_EC_CURVES too?

@bagder
Copy link
Member

bagder commented Oct 15, 2020

I would imagine it does!

This fixes the OS400 build and also an incorrect entry for
CURLINFO_APPCONNECT_TIME_T where it was treated as
CURLINFO_STARTTRANSFER_TIME_T.

Reported-by: Jon Rumsey

Fixes curl#6083
Closes #xxxx
@jay jay closed this in 3bcd81d Oct 18, 2020
@jay jay deleted the fix_os400_build branch October 18, 2020 02:44
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.

libCurl 7.73.0 CHKSTRINGS failure on OS400
3 participants