#3878 UTIL: Fix compilation with curl 7.62.0
Merged by jhrozek. Opened by lslebodn.
SSSD/ lslebodn/sssd curl_7_62_0  into  master

Download 3878.patch

The macro CURLE_SSL_CACERT is deprecated in upstream curl
since commit 3f3b26d6feb0667714902e836af608094235fca2.

commit 3f3b26d6feb0667714902e836af608094235fca2
Author: Han Han hhan@thousandeyes.com
Date: Wed Aug 22 11:13:32 2018 -0700

  ssl: deprecate CURLE_SSL_CACERT in favour of a unified error code
  Long live CURLE_PEER_FAILED_VERIFICATION

sh$ git tag --contains 3f3b26d6feb0667714902e836af608094235fca2
curl-7_62_0

It was not removed. It is just an alias to
CURLE_PEER_FAILED_VERIFICATION which causes compile time failures in
switch/case.

./src/util/tev_curl.c: In function 'curl_code2errno':
./src/util/tev_curl.c:113:5: error: duplicate case value
case CURLE_PEER_FAILED_VERIFICATION:
^~~~
./src/util/tev_curl.c: 100:5: note: previously used here
case CURLE_SSL_CACERT:
^~~~

Resolves:
https://pagure.io/SSSD/sssd/issue/3875

rebased onto 66344b8ef47e4ef5cfdafc1a3b28c5c28a3d3843

Thank you for the patch, the patch looks good and I agree that just using #if directly in the code the most straight forward solution. I'll do a CI run before giving the final ACK.

CI run was fine. ACK.

Commit 1ee12b05 fixes this pull-request

Pull-Request has been merged by jhrozek

Metadata