summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authororbea <orbea@riseup.net>2023-05-11 07:37:07 -0700
committerSam James <sam@gentoo.org>2023-05-11 22:05:16 +0100
commitc91cafb9e46fa580f698a54eecf1752aab89ab2a (patch)
tree2d46d0f2320962c1a84379e89dcbc788f7d8ecc5
parentsci-geosciences/josm-bin: drop old (diff)
downloadgentoo-c91cafb9e46fa580f698a54eecf1752aab89ab2a.tar.gz
gentoo-c91cafb9e46fa580f698a54eecf1752aab89ab2a.tar.bz2
gentoo-c91cafb9e46fa580f698a54eecf1752aab89ab2a.zip
net-libs/serf: fix dev-vcs/subversion build fail
With the previous backported LibreSSL patch the build in dev-vcs/subversion fails with an undefined reference to CRYPTO_malloc_init() which should be used with LibreSSL and not OpenSSL. To avoid this unintended issue the patch is changed to check only for LIBRESSL_VERSION_NUMBER. When the next serf release happens this patch can be removed and the new upstream cmake build system may be preferable. Closes: https://bugs.gentoo.org/906123 Signed-off-by: orbea <orbea@riseup.net> Closes: https://github.com/gentoo/gentoo/pull/30992 Signed-off-by: Sam James <sam@gentoo.org>
-rw-r--r--net-libs/serf/files/serf-1.3.9-libressl.patch28
-rw-r--r--net-libs/serf/serf-1.3.9-r4.ebuild (renamed from net-libs/serf/serf-1.3.9-r3.ebuild)2
2 files changed, 7 insertions, 23 deletions
diff --git a/net-libs/serf/files/serf-1.3.9-libressl.patch b/net-libs/serf/files/serf-1.3.9-libressl.patch
index 1c437e81bc05..bfb07bd5a0ad 100644
--- a/net-libs/serf/files/serf-1.3.9-libressl.patch
+++ b/net-libs/serf/files/serf-1.3.9-libressl.patch
@@ -1,3 +1,8 @@
+This backported patch avoids the upstream scons check to prevent unintended
+issues and can be removed next release bump where it may be preferable to
+use the newer cmake build system.
+
+https://bugs.gentoo.org/906123
https://bugs.gentoo.org/903001
https://github.com/apache/serf/commit/df0d2d0dbdf88576f26da9c71df3ab6249d351dc
@@ -14,28 +19,7 @@ Subject: [PATCH] Fix another build problem with LibreSSL.
of USE_LEGACY_OPENSSL.
git-svn-id: https://svn.apache.org/repos/asf/serf/trunk@1775242 13f79535-47bb-0310-9956-ffa450edef68
----
- SConstruct | 6 ++++++
- buckets/ssl_buckets.c | 2 +-
- 2 files changed, 7 insertions(+), 1 deletion(-)
-diff --git a/SConstruct b/SConstruct
-index 4358a23..115f409 100644
---- a/SConstruct
-+++ b/SConstruct
-@@ -373,6 +373,12 @@ else:
- env.Append(CPPPATH=['$OPENSSL/include'])
- env.Append(LIBPATH=['$OPENSSL/lib'])
-
-+ # Check for OpenSSL functions which are only available in some of
-+ # the versions we support. Also handles forks like LibreSSL.
-+ conf = Configure(env)
-+ if conf.CheckFunc('OPENSSL_malloc_init'):
-+ env.Append(CPPDEFINES=['SERF_HAVE_OPENSSL_MALLOC_INIT'])
-+ env = conf.Finish()
-
- # If build with gssapi, get its information and define SERF_HAVE_GSSAPI
- if gssapi and CALLOUT_OKAY:
diff --git a/buckets/ssl_buckets.c b/buckets/ssl_buckets.c
index b01e535..27c84c7 100644
--- a/buckets/ssl_buckets.c
@@ -45,7 +29,7 @@ index b01e535..27c84c7 100644
#endif
-#ifdef USE_OPENSSL_1_1_API
-+#ifdef SERF_HAVE_OPENSSL_MALLOC_INIT
++#ifndef LIBRESSL_VERSION_NUMBER
OPENSSL_malloc_init();
#else
CRYPTO_malloc_init();
diff --git a/net-libs/serf/serf-1.3.9-r3.ebuild b/net-libs/serf/serf-1.3.9-r4.ebuild
index 7464e14dba6d..8ea7ea9652d6 100644
--- a/net-libs/serf/serf-1.3.9-r3.ebuild
+++ b/net-libs/serf/serf-1.3.9-r4.ebuild
@@ -34,7 +34,7 @@ PATCHES=(
"${FILESDIR}"/${PN}-1.3.9-python3-check.patch
"${FILESDIR}"/${PN}-1.3.9-openssl-3-bio-ctrl.patch
"${FILESDIR}"/${PN}-1.3.9-openssl-3-errgetfunc.patch
- "${FILESDIR}"/${PN}-1.3.9-libressl.patch #903001
+ "${FILESDIR}"/${PN}-1.3.9-libressl.patch #903001, 906123
)
src_prepare() {