[ 
https://issues.apache.org/jira/browse/XERCESC-2156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17052450#comment-17052450
 ] 

Roger Leigh commented on XERCESC-2156:
--------------------------------------

The provided patch switches from the CMake FindCURL script to using pkg-config 
by default.  I'm not sure that's the correct approach, but would be happy to 
discuss further.

If FindCURL is not finding the static version of the CURL library, I think the 
best course of action would be to contribute the necessary fix for this module 
to CMake upstream and/or the CURL upstream if it's a CURL-provided config 
module.

The patch as provided is using pkg-config by default and then falling back to 
FindCURL as a fallback.  If we were to go with this approach, I think that we 
should be doing it the other way around and falling back to pkg-config, since 
FindCURL should be the canonical way to find it with CMake.

> fix static linking with curl
> ----------------------------
>
>                 Key: XERCESC-2156
>                 URL: https://issues.apache.org/jira/browse/XERCESC-2156
>             Project: Xerces-C++
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: 3.2.0, 3.2.1, 3.2.2
>            Reporter: Fabrice Fontaine
>            Assignee: Roger Leigh
>            Priority: Minor
>             Fix For: 3.2.3
>
>         Attachments: 0001-fix-static-linking-with-curl.patch
>
>
> When curl is statically built with openssl support, xerces needs to
> link with openssl libraries so use pkg_check_modules to get any
> needed dependencies
> Fixes:
>  - 
> http://autobuild.buildroot.org/results/29ca90fff2c8e38f2edf7240eca3aa3fe7397c45



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscr...@xerces.apache.org
For additional commands, e-mail: c-dev-h...@xerces.apache.org

Reply via email to