Skip to contentSkip to conversation

xerces: fix static linking with curl

PATCH New Fabrice Fontaine 2 participants 2 messages Oct 11, 2018 – Oct 14, 2018

lore
Share & tools
lore

Fabrice Fontaine

author
#0 lore raw

PATCH 1/1 xerces: fix static linking with curl

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

Signed-off-by Fabrice Fontaine <[email protected]>
---
 .../0002-fix-static-linking-with-curl.patch   | 39 +++++++++++++++++++
 1 file changed, 39 insertions(+)
 create mode 100644 package/xerces/0002-fix-static-linking-with-curl.patch
package/xerces/0002-fix-static-linking-with-curl.patch new file +39 -0
new file mode 100644
index 0000000000..db38de79fc
@@ -0,0 +1,39 @@
+From 26e3f1440bbc75c704fc93ff43a2abbfbe4c0203 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <[email protected]>
+Date: Thu, 11 Oct 2018 22:56:50 +0200
+Subject: [PATCH] fix static linking with curl
+
+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
+
+Signed-off-by: Fabrice Fontaine <[email protected]>
+---
+ cmake/XercesNetAccessorSelection.cmake | 8 +++++++-
+ 1 file changed, 7 insertions(+), 1 deletion(-)
+
+diff --git a/cmake/XercesNetAccessorSelection.cmake b/cmake/XercesNetAccessorSelection.cmake
+index 7a63f1f6b..e90fcf034 100644
+--- a/cmake/XercesNetAccessorSelection.cmake
++++ b/cmake/XercesNetAccessorSelection.cmake
+@@ -31,7 +31,13 @@ if(network)
+ 
+   # Requires select() which is UNIX only
+   if(UNIX)
+-    find_package(CURL)
++    find_package(PkgConfig)
++    if (PkgConfig_FOUND)
++      pkg_check_modules(CURL libcurl)
++      if (NOT CURL_FOUND)
++        find_package(CURL)
++      endif()
++    endif()
+     if(CURL_FOUND)
+       list(APPEND netaccessors curl)
+     endif()
+-- 
+2.17.1
+
Signature · 1 line
2.17.1
Applied
#1 lore raw

PATCH 1/1 xerces: fix static linking with curl

Hello,

Quoted 4 lines · Fabrice Fontaine

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

Quoted 8 lines

Fixes:
- http://autobuild.buildroot.org/results/29ca90fff2c8e38f2edf7240eca3aa3fe7397c45

Signed-off-by Fabrice Fontaine <[email protected]>
---
 .../0002-fix-static-linking-with-curl.patch   | 39 +++++++++++++++++++
 1 file changed, 39 insertions(+)
 create mode 100644 package/xerces/0002-fix-static-linking-with-curl.patch

Ah, an upstreamable pkg-config based solution to solve static linking
issues: good! Could you make sure to submit this upstream?

Applied to master. Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com