From: dkwo <dkwo@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] thrift: update (openssl3)
Date: Mon, 06 Mar 2023 19:20:03 +0100 [thread overview]
Message-ID: <20230306182003.PJkV3khXLcgQMVIH7m3xfAA4-wiUJMHCax-Qz7RLy4Y@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-41967@inbox.vuxu.org>
[-- Attachment #1: Type: text/plain, Size: 459 bytes --]
There is an updated pull request by dkwo against master on the void-packages repository
https://github.com/dkwo/void-packages O3thrift
https://github.com/void-linux/void-packages/pull/41967
thrift: update (openssl3)
- I tested the changes in this PR: no
- I built this PR locally for my native architecture, (x86_64-musl)
required to build with openssl3 #37681
A patch file from https://github.com/void-linux/void-packages/pull/41967.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-O3thrift-41967.patch --]
[-- Type: text/x-diff, Size: 5970 bytes --]
From 5257370cc7934a6b35b104b7150f46eb65402ab2 Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Thu, 26 Jan 2023 14:03:20 -0500
Subject: [PATCH] thrift: update to 0.17.0, patch for openssl3
---
srcpkgs/thrift/patches/missing-signal-h.patch | 17 +++++++
srcpkgs/thrift/patches/musl-libc-locale.patch | 24 ++++++++++
.../thrift/patches/skip-old-tls-tests.patch | 44 +++++++++++++++++++
srcpkgs/thrift/template | 24 ++++++----
4 files changed, 100 insertions(+), 9 deletions(-)
create mode 100644 srcpkgs/thrift/patches/missing-signal-h.patch
create mode 100644 srcpkgs/thrift/patches/musl-libc-locale.patch
create mode 100644 srcpkgs/thrift/patches/skip-old-tls-tests.patch
diff --git a/srcpkgs/thrift/patches/missing-signal-h.patch b/srcpkgs/thrift/patches/missing-signal-h.patch
new file mode 100644
index 000000000000..ec1ea196b0e8
--- /dev/null
+++ b/srcpkgs/thrift/patches/missing-signal-h.patch
@@ -0,0 +1,17 @@
+From: https://git.alpinelinux.org/aports/plain/community/thrift/missing-signal-h.patch
+
+Add missing include for signal.h
+--- a/lib/cpp/test/TNonblockingSSLServerTest.cpp
++++ b/lib/cpp/test/TNonblockingSSLServerTest.cpp
+@@ -30,6 +30,10 @@
+
+ #include <event.h>
+
++#ifdef __linux__
++#include <signal.h>
++#endif
++
+ using namespace apache::thrift;
+ using apache::thrift::concurrency::Guard;
+ using apache::thrift::concurrency::Monitor;
+
diff --git a/srcpkgs/thrift/patches/musl-libc-locale.patch b/srcpkgs/thrift/patches/musl-libc-locale.patch
new file mode 100644
index 000000000000..2480c687d6dc
--- /dev/null
+++ b/srcpkgs/thrift/patches/musl-libc-locale.patch
@@ -0,0 +1,24 @@
+From: https://git.alpinelinux.org/aports/plain/community/thrift/musl-libc-locale.patch
+
+Skip tests using locale functionality
+--- a/lib/cpp/test/ToStringTest.cpp
++++ b/lib/cpp/test/ToStringTest.cpp
+@@ -49,7 +49,7 @@ BOOST_AUTO_TEST_CASE(locale_en_US_int_to_string) {
+ #ifdef _WIN32
+ std::locale::global(std::locale("en-US.UTF-8"));
+ #else
+- std::locale::global(std::locale("en_US.UTF-8"));
++ // musl libc has no locale support
+ #endif
+ BOOST_CHECK_EQUAL(to_string(1000000), "1000000");
+ }
+@@ -58,7 +58,7 @@ BOOST_AUTO_TEST_CASE(locale_de_DE_floating_point_to_string) {
+ #ifdef _WIN32
+ std::locale::global(std::locale("de-DE.UTF-8"));
+ #else
+- std::locale::global(std::locale("de_DE.UTF-8"));
++ // musl libc has no locale support
+ #endif
+ BOOST_CHECK_EQUAL(to_string(1.5), "1.5");
+ BOOST_CHECK_EQUAL(to_string(1.5f), "1.5");
+
diff --git a/srcpkgs/thrift/patches/skip-old-tls-tests.patch b/srcpkgs/thrift/patches/skip-old-tls-tests.patch
new file mode 100644
index 000000000000..dafb5abe6e28
--- /dev/null
+++ b/srcpkgs/thrift/patches/skip-old-tls-tests.patch
@@ -0,0 +1,44 @@
+From: https://git.alpinelinux.org/aports/plain/community/thrift/skip-old-tls-tests.patch
+
+Skip TLSv1_0 and TLSv1_1 test cases
+--- a/lib/cpp/test/SecurityFromBufferTest.cpp
++++ b/lib/cpp/test/SecurityFromBufferTest.cpp
+@@ -224,6 +224,16 @@ BOOST_AUTO_TEST_CASE(ssl_security_matrix) {
+ }
+ #endif
+
++ if (si == 3 || ci == 3) {
++ // Skip all TLSv1_0 cases - protocol not supported
++ continue;
++ }
++
++ if (si == 4 || ci == 4) {
++ // Skip all TLSv1_1 cases - protocol not supported
++ continue;
++ }
++
+ boost::mutex::scoped_lock lock(mMutex);
+
+ BOOST_TEST_MESSAGE(boost::format("TEST: Server = %1%, Client = %2%") % protocol2str(si)
+--- a/lib/cpp/test/SecurityTest.cpp
++++ b/lib/cpp/test/SecurityTest.cpp
+@@ -250,6 +250,18 @@ BOOST_AUTO_TEST_CASE(ssl_security_matrix)
+ }
+ #endif
+
++ if (si == 3 || ci == 3)
++ {
++ // Skip all TLSv1_0 cases - protocol not supported
++ continue;
++ }
++
++ if (si == 4 || ci == 4)
++ {
++ // Skip all TLSv1_1 cases - protocol not supported
++ continue;
++ }
++
+ boost::mutex::scoped_lock lock(mMutex);
+
+ BOOST_TEST_MESSAGE(boost::format("TEST: Server = %1%, Client = %2%")
+
diff --git a/srcpkgs/thrift/template b/srcpkgs/thrift/template
index 9ac4bb481c30..2b4b29b48b84 100644
--- a/srcpkgs/thrift/template
+++ b/srcpkgs/thrift/template
@@ -1,31 +1,37 @@
# Template file for 'thrift'
pkgname=thrift
-version=0.13.0
-revision=6
-build_style=gnu-configure
-configure_args="--without-python"
+version=0.17.0
+revision=1
+build_style=cmake
+configure_args="-DBUILD_SHARED_LIBS=ON
+ -DWITH_AS3=OFF
+ -DWITH_JAVA=OFF
+ -DWITH_JAVASCRIPT=OFF
+ -DWITH_NODEJS=OFF
+ -DWITH_PYTHON=OFF"
+hostmakedepends="flex"
makedepends="boost-devel openssl-devel"
checkdepends="zlib-devel libevent-devel"
short_desc="Apache Thrift compiler"
maintainer="Orphaned <orphan@voidlinux.org>"
license="Apache-2.0"
homepage="https://thrift.apache.org/"
-distfiles="http://www-us.apache.org/dist/thrift/${version}/thrift-${version}.tar.gz"
-checksum=7ad348b88033af46ce49148097afe354d513c1fca7c607b59c33ebb6064b5179
+distfiles="https://github.com/apache/thrift/archive/v${version}.tar.gz"
+checksum=f5888bcd3b8de40c2c2ab86896867ad9b18510deb412cba3e5da76fb4c604c29
if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
makedepends+=" libatomic-devel"
fi
if [ "$CROSS_BUILD" ]; then
- configure_args+=" --disable-tests --disable-tutorial"
+ configure_args+=" -DBUILD_TESTING=OFF -DBUILD_TUTORIALS=OFF"
fi
post_configure() {
case "$XBPS_TARGET_MACHINE" in
ppc64*) ;;
ppc*)
- sed -i '/libthrift_la_LIBADD/s/$/ -latomic/' lib/cpp/Makefile
+ vsed -i '/libthrift_la_LIBADD/s/$/ -latomic/' lib/cpp/Makefile
;;
esac
}
@@ -35,8 +41,8 @@ thrift-devel_package() {
depends="${sourcepkg}>=${version}_${revision}"
pkg_install() {
vmove usr/include
- vmove "usr/lib/*.a"
vmove "usr/lib/*.so"
+ vmove usr/lib/cmake
vmove usr/lib/pkgconfig
}
}
next prev parent reply other threads:[~2023-03-06 18:20 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
2023-01-30 17:08 ` [PR PATCH] [Updated] " dkwo
2023-01-30 17:12 ` dkwo
2023-01-30 17:13 ` dkwo
2023-01-30 18:15 ` [PR REVIEW] " Duncaen
2023-01-30 18:16 ` Duncaen
2023-01-30 18:18 ` [PR PATCH] [Updated] " dkwo
2023-01-30 18:18 ` [PR REVIEW] " dkwo
2023-01-30 18:21 ` [PR PATCH] [Updated] " dkwo
2023-01-30 18:22 ` dkwo
2023-02-01 16:10 ` [PR REVIEW] thrift: update (openssl3) kruceter
2023-02-01 16:10 ` kruceter
2023-02-01 16:10 ` kruceter
2023-02-01 16:12 ` Duncaen
2023-02-01 16:33 ` dkwo
2023-02-01 16:33 ` [PR REVIEW] " dkwo
2023-02-01 16:34 ` dkwo
2023-02-01 16:35 ` [PR PATCH] [Updated] " dkwo
2023-02-01 16:35 ` dkwo
2023-02-03 23:20 ` kruceter
2023-02-03 23:26 ` [PR REVIEW] " paper42
2023-02-04 16:26 ` [PR PATCH] [Updated] " dkwo
2023-02-04 16:27 ` [PR REVIEW] " dkwo
2023-02-05 15:43 ` [PR PATCH] [Updated] " dkwo
2023-02-05 18:05 ` dkwo
2023-03-06 18:20 ` dkwo [this message]
2023-03-14 21:18 ` dkwo
2023-06-02 15:16 ` [PR PATCH] [Updated] " dkwo
2023-06-02 15:58 ` dkwo
2023-06-02 16:04 ` [PR PATCH] [Updated] " dkwo
2023-06-02 16:13 ` dkwo
2023-06-28 20:42 ` dkwo
2023-06-28 20:48 ` Bnyro
2023-06-28 20:52 ` [PR PATCH] [Merged]: " Duncaen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230306182003.PJkV3khXLcgQMVIH7m3xfAA4-wiUJMHCax-Qz7RLy4Y@z \
--to=dkwo@users.noreply.github.com \
--cc=ml@inbox.vuxu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).