* [PR PATCH] thrift: update distfiles, bump for openssl
@ 2023-01-30 15:26 dkwo
2023-01-30 17:08 ` [PR PATCH] [Updated] " dkwo
` (32 more replies)
0 siblings, 33 replies; 34+ messages in thread
From: dkwo @ 2023-01-30 15:26 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 471 bytes --]
There is a new 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 distfiles, bump for openssl
- 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: 1119 bytes --]
From 13d5465bd740f94a66ab558eb91ed4c1c6262c3d 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 distfiles, bump for openssl
---
srcpkgs/thrift/template | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/thrift/template b/srcpkgs/thrift/template
index 9ac4bb481c30..8b1a0c3247fd 100644
--- a/srcpkgs/thrift/template
+++ b/srcpkgs/thrift/template
@@ -1,7 +1,7 @@
# Template file for 'thrift'
pkgname=thrift
version=0.13.0
-revision=6
+revision=7
build_style=gnu-configure
configure_args="--without-python"
makedepends="boost-devel openssl-devel"
@@ -10,7 +10,7 @@ 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"
+distfiles="https://archive.apache.org/dist/thrift/${version}/thrift-${version}.tar.gz"
checksum=7ad348b88033af46ce49148097afe354d513c1fca7c607b59c33ebb6064b5179
if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update distfiles, bump for openssl
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
@ 2023-01-30 17:08 ` dkwo
2023-01-30 17:12 ` dkwo
` (31 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-01-30 17:08 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 476 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 distfiles, bump for openssl
- 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: 5536 bytes --]
From ff8a3afd38bcc584191292f8076007452dbe6eaa 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, patch
---
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 | 21 +++++----
4 files changed, 98 insertions(+), 8 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..304eb047b2bb 100644
--- a/srcpkgs/thrift/template
+++ b/srcpkgs/thrift/template
@@ -1,24 +1,29 @@
# Template file for 'thrift'
pkgname=thrift
-version=0.13.0
-revision=6
-build_style=gnu-configure
-configure_args="--without-python"
-makedepends="boost-devel openssl-devel"
+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"
+makedepends="boost-devel openssl-devel flex"
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() {
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update distfiles, bump for openssl
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
` (30 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-01-30 17:12 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 476 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 distfiles, bump for openssl
- 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: 5561 bytes --]
From 8aade36870851de3c404c39ce69ecb0b44aa3819 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, patch
---
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 | 22 ++++++----
4 files changed, 99 insertions(+), 8 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..184e81816879 100644
--- a/srcpkgs/thrift/template
+++ b/srcpkgs/thrift/template
@@ -1,24 +1,30 @@
# Template file for 'thrift'
pkgname=thrift
-version=0.13.0
-revision=6
-build_style=gnu-configure
-configure_args="--without-python"
-makedepends="boost-devel openssl-devel"
+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 flex"
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() {
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update distfiles, bump for openssl
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
` (29 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-01-30 17:13 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 157 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1409005306
Comment:
actually update it
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update distfiles, bump for openssl
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (2 preceding siblings ...)
2023-01-30 17:13 ` dkwo
@ 2023-01-30 18:15 ` Duncaen
2023-01-30 18:16 ` Duncaen
` (28 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: Duncaen @ 2023-01-30 18:15 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 173 bytes --]
New review comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1090983051
Comment:
indent with single space.
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update distfiles, bump for openssl
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (3 preceding siblings ...)
2023-01-30 18:15 ` [PR REVIEW] " Duncaen
@ 2023-01-30 18:16 ` Duncaen
2023-01-30 18:18 ` [PR PATCH] [Updated] " dkwo
` (27 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: Duncaen @ 2023-01-30 18:16 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 183 bytes --]
New review comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1090983285
Comment:
`\` is not needed in double quotes.
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update distfiles, bump for openssl
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (4 preceding siblings ...)
2023-01-30 18:16 ` Duncaen
@ 2023-01-30 18:18 ` dkwo
2023-01-30 18:18 ` [PR REVIEW] " dkwo
` (26 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-01-30 18:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 476 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 distfiles, bump for openssl
- 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: 5770 bytes --]
From c591c26dda9113a249c121eaf759e7cc571ea2ea 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, patch
---
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 | 23 ++++++----
4 files changed, 99 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..de9a6f5f4282 100644
--- a/srcpkgs/thrift/template
+++ b/srcpkgs/thrift/template
@@ -1,24 +1,30 @@
# Template file for 'thrift'
pkgname=thrift
-version=0.13.0
-revision=6
-build_style=gnu-configure
-configure_args="--without-python"
-makedepends="boost-devel openssl-devel"
+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 flex"
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() {
@@ -35,7 +41,6 @@ thrift-devel_package() {
depends="${sourcepkg}>=${version}_${revision}"
pkg_install() {
vmove usr/include
- vmove "usr/lib/*.a"
vmove "usr/lib/*.so"
vmove usr/lib/pkgconfig
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update distfiles, bump for openssl
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (5 preceding siblings ...)
2023-01-30 18:18 ` [PR PATCH] [Updated] " dkwo
@ 2023-01-30 18:18 ` dkwo
2023-01-30 18:21 ` [PR PATCH] [Updated] " dkwo
` (25 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-01-30 18:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 150 bytes --]
New review comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1090986346
Comment:
done.
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update distfiles, bump for openssl
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (6 preceding siblings ...)
2023-01-30 18:18 ` [PR REVIEW] " dkwo
@ 2023-01-30 18:21 ` dkwo
2023-01-30 18:22 ` dkwo
` (24 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-01-30 18:21 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 476 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 distfiles, bump for openssl
- 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: 5760 bytes --]
From 427a380246040766ac5730b417e70c0ac16b9b11 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, patch
---
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 | 23 ++++++----
4 files changed, 99 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..9d4d10deaa37 100644
--- a/srcpkgs/thrift/template
+++ b/srcpkgs/thrift/template
@@ -1,24 +1,30 @@
# Template file for 'thrift'
pkgname=thrift
-version=0.13.0
-revision=6
-build_style=gnu-configure
-configure_args="--without-python"
-makedepends="boost-devel openssl-devel"
+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 flex"
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() {
@@ -35,7 +41,6 @@ thrift-devel_package() {
depends="${sourcepkg}>=${version}_${revision}"
pkg_install() {
vmove usr/include
- vmove "usr/lib/*.a"
vmove "usr/lib/*.so"
vmove usr/lib/pkgconfig
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update distfiles, bump for openssl
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (7 preceding siblings ...)
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
` (23 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-01-30 18:22 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 476 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 distfiles, bump for openssl
- 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: 5760 bytes --]
From b42f89e3974e42d4501d74a47974ce230b3eb868 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, patch
---
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 | 23 ++++++----
4 files changed, 99 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..c89c5a506eaf 100644
--- a/srcpkgs/thrift/template
+++ b/srcpkgs/thrift/template
@@ -1,24 +1,30 @@
# Template file for 'thrift'
pkgname=thrift
-version=0.13.0
-revision=6
-build_style=gnu-configure
-configure_args="--without-python"
-makedepends="boost-devel openssl-devel"
+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 flex"
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=54c30b6ac27d62aee29e8d07831c1c785bf7ed8cd351e84130abe7d4f564335d
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() {
@@ -35,7 +41,6 @@ thrift-devel_package() {
depends="${sourcepkg}>=${version}_${revision}"
pkg_install() {
vmove usr/include
- vmove "usr/lib/*.a"
vmove "usr/lib/*.so"
vmove usr/lib/pkgconfig
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (8 preceding siblings ...)
2023-01-30 18:22 ` dkwo
@ 2023-02-01 16:10 ` kruceter
2023-02-01 16:10 ` kruceter
` (22 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: kruceter @ 2023-02-01 16:10 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 311 bytes --]
New review comment by kruceter on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1093420825
Comment:
Is `flex` really necessary for `makedepends`?
At first I assumed it has to do something with cross, but alas - it builds with just `flex` in `hostmakedepends`.
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (9 preceding siblings ...)
2023-02-01 16:10 ` [PR REVIEW] thrift: update (openssl3) kruceter
@ 2023-02-01 16:10 ` kruceter
2023-02-01 16:10 ` kruceter
` (21 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: kruceter @ 2023-02-01 16:10 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 257 bytes --]
New review comment by kruceter on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1093418253
Comment:
Please triple-check the checkum value because it mismatched locally. Github *loves* to make life unbearable.
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (10 preceding siblings ...)
2023-02-01 16:10 ` kruceter
@ 2023-02-01 16:10 ` kruceter
2023-02-01 16:12 ` Duncaen
` (20 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: kruceter @ 2023-02-01 16:10 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 398 bytes --]
New review comment by kruceter on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1093427537
Comment:
```suggestion
configure_args="-DBUILD_SHARED_LIBS=ON -DWITH_AS3=OFF -DWITH_JAVA=OFF
-DWITH_JAVASCRIPT=OFF -DWITH_NODEJS=OFF -DWITH_PYTHON=OFF"
```
I do not think every option should be splattered one per line; correct me if I am wrong though.
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (11 preceding siblings ...)
2023-02-01 16:10 ` kruceter
@ 2023-02-01 16:12 ` Duncaen
2023-02-01 16:33 ` dkwo
` (19 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: Duncaen @ 2023-02-01 16:12 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 181 bytes --]
New review comment by Duncaen on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1093439250
Comment:
This is fine imho, I do the same.
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (12 preceding siblings ...)
2023-02-01 16:12 ` Duncaen
@ 2023-02-01 16:33 ` dkwo
2023-02-01 16:33 ` [PR REVIEW] " dkwo
` (18 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-01 16:33 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 343 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1412356489
Comment:
> There is also a `sed` command in the template which should be replaced to `vsed`, but I myself am not sure who ever build on ppc64* these days.
I'll do as you say, but I did not check the ppc builds.
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (13 preceding siblings ...)
2023-02-01 16:33 ` dkwo
@ 2023-02-01 16:33 ` dkwo
2023-02-01 16:34 ` dkwo
` (17 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-01 16:33 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 159 bytes --]
New review comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1093465196
Comment:
ok, let me try
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (14 preceding siblings ...)
2023-02-01 16:33 ` [PR REVIEW] " dkwo
@ 2023-02-01 16:34 ` dkwo
2023-02-01 16:35 ` [PR PATCH] [Updated] " dkwo
` (16 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-01 16:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 208 bytes --]
New review comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1093466195
Comment:
yes, github changed something two or more times the other day..
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (15 preceding siblings ...)
2023-02-01 16:34 ` dkwo
@ 2023-02-01 16:35 ` dkwo
2023-02-01 16:35 ` dkwo
` (15 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-01 16:35 UTC (permalink / raw)
To: ml
[-- 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: 5943 bytes --]
From 33490b743842fb29e78dd3a5ae8f3d1ea6487846 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 | 23 ++++++----
4 files changed, 99 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..03ea5ee334fa 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,7 +41,6 @@ thrift-devel_package() {
depends="${sourcepkg}>=${version}_${revision}"
pkg_install() {
vmove usr/include
- vmove "usr/lib/*.a"
vmove "usr/lib/*.so"
vmove usr/lib/pkgconfig
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (16 preceding siblings ...)
2023-02-01 16:35 ` [PR PATCH] [Updated] " dkwo
@ 2023-02-01 16:35 ` dkwo
2023-02-03 23:20 ` kruceter
` (14 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-01 16:35 UTC (permalink / raw)
To: ml
[-- 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: 5943 bytes --]
From b6ea3e9140018de31561ff4dd5862c4deb3bd494 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 | 23 ++++++----
4 files changed, 99 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..03ea5ee334fa 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,7 +41,6 @@ thrift-devel_package() {
depends="${sourcepkg}>=${version}_${revision}"
pkg_install() {
vmove usr/include
- vmove "usr/lib/*.a"
vmove "usr/lib/*.so"
vmove usr/lib/pkgconfig
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (17 preceding siblings ...)
2023-02-01 16:35 ` dkwo
@ 2023-02-03 23:20 ` kruceter
2023-02-03 23:26 ` [PR REVIEW] " paper42
` (13 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: kruceter @ 2023-02-03 23:20 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 166 bytes --]
New comment by kruceter on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1416519568
Comment:
@Vaelatern, if you may?
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (18 preceding siblings ...)
2023-02-03 23:20 ` kruceter
@ 2023-02-03 23:26 ` paper42
2023-02-04 16:26 ` [PR PATCH] [Updated] " dkwo
` (12 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: paper42 @ 2023-02-03 23:26 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 216 bytes --]
New review comment by paper42 on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1096377901
Comment:
```suggestion
vmove usr/lib/cmake
vmove usr/lib/pkgconfig
```
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (19 preceding siblings ...)
2023-02-03 23:26 ` [PR REVIEW] " paper42
@ 2023-02-04 16:26 ` dkwo
2023-02-04 16:27 ` [PR REVIEW] " dkwo
` (11 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-04 16:26 UTC (permalink / raw)
To: ml
[-- 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 d1ba2cb019afa5e05934473bf2cf44f8987938e6 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
}
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR REVIEW] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (20 preceding siblings ...)
2023-02-04 16:26 ` [PR PATCH] [Updated] " dkwo
@ 2023-02-04 16:27 ` dkwo
2023-02-05 15:43 ` [PR PATCH] [Updated] " dkwo
` (10 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-04 16:27 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 149 bytes --]
New review comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#discussion_r1096556513
Comment:
Done
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (21 preceding siblings ...)
2023-02-04 16:27 ` [PR REVIEW] " dkwo
@ 2023-02-05 15:43 ` dkwo
2023-02-05 18:05 ` dkwo
` (9 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-05 15:43 UTC (permalink / raw)
To: ml
[-- 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 f84da687c4863e421eb7439ccbeca81f38802a2f 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
}
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (22 preceding siblings ...)
2023-02-05 15:43 ` [PR PATCH] [Updated] " dkwo
@ 2023-02-05 18:05 ` dkwo
2023-03-06 18:20 ` [PR PATCH] [Updated] " dkwo
` (8 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-02-05 18:05 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1702 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1418203294
Comment:
This test was passing before on x86:
```
383/408 Test #383: TServerIntegrationTest ...........***Exception: SegFault 3.85 sec
terminate called after throwing an instance of 'std::system_error'
what(): Resource temporarily unavailable
Running 13 test cases...
unknown location(0): fatal error: in "constructors/test_threaded_stress": signal: SIGABRT (application abort requested)
/builddir/thrift-0.17.0/lib/cpp/test/TServerIntegrationTest.cpp(373): last checkpoint: "test_threaded_stress" test entry
Thrift: Sun Feb 5 15:50:46 2023 TSocket::open() connect() <Host: 127.0.0.1 Port: 32931>: Connection refused
Thrift: Sun Feb 5 15:50:46 2023 TSocket::open() connect() <Host: 127.0.0.1 Port: 32931>: Connection refused
terminate called recursively
unknown location(0): fatal error: in "constructors/test_threadpool_factory": signal: SIGABRT (application abort requested)
/builddir/thrift-0.17.0/lib/cpp/test/TServerIntegrationTest.cpp(378): last checkpoint: "test_threadpool_factory" test entry
Thrift: Sun Feb 5 15:50:46 2023 TSocket::open() connect() <Host: 127.0.0.1 Port: 32931>: Connection refused
Thrift: Sun Feb 5 15:50:46 2023 TSocket::open() connect() <Host: 127.0.0.1 Port: 32931>: Connection refused
terminate called recursively
unknown location(0): fatal error: in "constructors/test_threadpool": signal: SIGABRT (application abort requested)
/builddir/thrift-0.17.0/lib/cpp/test/TServerIntegrationTest.cpp(393): last checkpoint: "test_threadpool" test entry
```
Could the failure be triggered by some change in the containers?
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (23 preceding siblings ...)
2023-02-05 18:05 ` dkwo
@ 2023-03-06 18:20 ` dkwo
2023-03-14 21:18 ` dkwo
` (7 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-03-06 18:20 UTC (permalink / raw)
To: ml
[-- 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
}
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (24 preceding siblings ...)
2023-03-06 18:20 ` [PR PATCH] [Updated] " dkwo
@ 2023-03-14 21:18 ` dkwo
2023-06-02 15:16 ` [PR PATCH] [Updated] " dkwo
` (6 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-03-14 21:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1702 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1418203294
Comment:
This test was passing before on x86:
```
383/408 Test #383: TServerIntegrationTest ...........***Exception: SegFault 3.85 sec
terminate called after throwing an instance of 'std::system_error'
what(): Resource temporarily unavailable
Running 13 test cases...
unknown location(0): fatal error: in "constructors/test_threaded_stress": signal: SIGABRT (application abort requested)
/builddir/thrift-0.17.0/lib/cpp/test/TServerIntegrationTest.cpp(373): last checkpoint: "test_threaded_stress" test entry
Thrift: Sun Feb 5 15:50:46 2023 TSocket::open() connect() <Host: 127.0.0.1 Port: 32931>: Connection refused
Thrift: Sun Feb 5 15:50:46 2023 TSocket::open() connect() <Host: 127.0.0.1 Port: 32931>: Connection refused
terminate called recursively
unknown location(0): fatal error: in "constructors/test_threadpool_factory": signal: SIGABRT (application abort requested)
/builddir/thrift-0.17.0/lib/cpp/test/TServerIntegrationTest.cpp(378): last checkpoint: "test_threadpool_factory" test entry
Thrift: Sun Feb 5 15:50:46 2023 TSocket::open() connect() <Host: 127.0.0.1 Port: 32931>: Connection refused
Thrift: Sun Feb 5 15:50:46 2023 TSocket::open() connect() <Host: 127.0.0.1 Port: 32931>: Connection refused
terminate called recursively
unknown location(0): fatal error: in "constructors/test_threadpool": signal: SIGABRT (application abort requested)
/builddir/thrift-0.17.0/lib/cpp/test/TServerIntegrationTest.cpp(393): last checkpoint: "test_threadpool" test entry
```
Could the failure be triggered by some change in the containers?
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (25 preceding siblings ...)
2023-03-14 21:18 ` dkwo
@ 2023-06-02 15:16 ` dkwo
2023-06-02 15:58 ` dkwo
` (5 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-06-02 15:16 UTC (permalink / raw)
To: ml
[-- 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: 3920 bytes --]
From d5e8d9da70d7e0981dc954f71c51209b6f6273c3 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.18.1, patch out old tls tests
---
.../thrift/patches/skip-old-tls-tests.patch | 44 +++++++++++++++++++
srcpkgs/thrift/template | 24 ++++++----
2 files changed, 59 insertions(+), 9 deletions(-)
create mode 100644 srcpkgs/thrift/patches/skip-old-tls-tests.patch
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..1912cfb7ed33 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.18.1
+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=9cea30b9700153329ae1926cc05a20bbe3e8451ae270b0c8c5c5fe9929924cb3
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
}
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (26 preceding siblings ...)
2023-06-02 15:16 ` [PR PATCH] [Updated] " dkwo
@ 2023-06-02 15:58 ` dkwo
2023-06-02 16:04 ` [PR PATCH] [Updated] " dkwo
` (4 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-06-02 15:58 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 797 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1573963937
Comment:
updated to 0.18, removed patches, now one glibc test fails:
```
17/44 Test #17: UnitTests ........................***Failed 1.32 sec
Running 60 test cases...
unknown location(0): fatal error: in "ToStringTest/locale_de_DE_floating_point_to_string": std::runtime_error: locale::facet::_S_create_c_locale name not valid
/builddir/thrift-0.18.1/lib/cpp/test/ToStringTest.cpp(57): last checkpoint: "locale_de_DE_floating_point_to_string" test entry
An error message from getaddrinfo on the console is expected:
Thrift: Fri Jun 2 15:23:52 2023 getaddrinfo() -> -2; Name or service not known
*** 1 failure is detected in the test module "thrift"
```
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Updated] thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (27 preceding siblings ...)
2023-06-02 15:58 ` dkwo
@ 2023-06-02 16:04 ` dkwo
2023-06-02 16:13 ` dkwo
` (3 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-06-02 16:04 UTC (permalink / raw)
To: ml
[-- 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: 5631 bytes --]
From 42856c8606edaaee4b6266f9b2778babd284b729 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.18.1, patch out old tls and broken tests
---
srcpkgs/thrift/patches/network-tests.patch | 39 ++++++++++++++++
.../thrift/patches/skip-old-tls-tests.patch | 44 +++++++++++++++++++
srcpkgs/thrift/template | 24 ++++++----
3 files changed, 98 insertions(+), 9 deletions(-)
create mode 100644 srcpkgs/thrift/patches/network-tests.patch
create mode 100644 srcpkgs/thrift/patches/skip-old-tls-tests.patch
diff --git a/srcpkgs/thrift/patches/network-tests.patch b/srcpkgs/thrift/patches/network-tests.patch
new file mode 100644
index 000000000000..49aa667752b7
--- /dev/null
+++ b/srcpkgs/thrift/patches/network-tests.patch
@@ -0,0 +1,39 @@
+From https://gitweb.gentoo.org/repo/gentoo.git/tree/dev-libs/thrift/files/thrift-0.16.0-network-tests.patch
+
+diff --git a/lib/cpp/test/ToStringTest.cpp b/lib/cpp/test/ToStringTest.cpp
+index 736b33c0a..f66909873 100644
+--- a/lib/cpp/test/ToStringTest.cpp
++++ b/lib/cpp/test/ToStringTest.cpp
+@@ -41,31 +41,6 @@ BOOST_AUTO_TEST_CASE(base_types_to_string) {
+ BOOST_CHECK_EQUAL(to_string("abc"), "abc");
+ }
+
+-// NOTE: Currently (as of 2021.08.12) the locale-based tests do not work on
+-// Windows in the AppVeyor Thrift CI build correctly. Therefore disabled on
+-// Windows:
+-#ifndef _WIN32
+-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"));
+-#endif
+- BOOST_CHECK_EQUAL(to_string(1000000), "1000000");
+-}
+-
+-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"));
+-#endif
+- BOOST_CHECK_EQUAL(to_string(1.5), "1.5");
+- BOOST_CHECK_EQUAL(to_string(1.5f), "1.5");
+- BOOST_CHECK_EQUAL(to_string(1.5L), "1.5");
+-}
+-#endif
+-
+ BOOST_AUTO_TEST_CASE(empty_vector_to_string) {
+ std::vector<int> l;
+ BOOST_CHECK_EQUAL(to_string(l), "[]");
+
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..1912cfb7ed33 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.18.1
+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=9cea30b9700153329ae1926cc05a20bbe3e8451ae270b0c8c5c5fe9929924cb3
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
}
}
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (28 preceding siblings ...)
2023-06-02 16:04 ` [PR PATCH] [Updated] " dkwo
@ 2023-06-02 16:13 ` dkwo
2023-06-28 20:42 ` dkwo
` (2 subsequent siblings)
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-06-02 16:13 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 150 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1573984911
Comment:
working now
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (29 preceding siblings ...)
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
32 siblings, 0 replies; 34+ messages in thread
From: dkwo @ 2023-06-28 20:42 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 167 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1612078121
Comment:
@Duncaen Can this be merged?
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (30 preceding siblings ...)
2023-06-28 20:42 ` dkwo
@ 2023-06-28 20:48 ` Bnyro
2023-06-28 20:52 ` [PR PATCH] [Merged]: " Duncaen
32 siblings, 0 replies; 34+ messages in thread
From: Bnyro @ 2023-06-28 20:48 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 213 bytes --]
New comment by Bnyro on void-packages repository
https://github.com/void-linux/void-packages/pull/41967#issuecomment-1612086077
Comment:
(also addresses https://github.com/void-linux/void-packages/issues/42403)
^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PR PATCH] [Merged]: thrift: update (openssl3)
2023-01-30 15:26 [PR PATCH] thrift: update distfiles, bump for openssl dkwo
` (31 preceding siblings ...)
2023-06-28 20:48 ` Bnyro
@ 2023-06-28 20:52 ` Duncaen
32 siblings, 0 replies; 34+ messages in thread
From: Duncaen @ 2023-06-28 20:52 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 308 bytes --]
There's a merged pull request on the void-packages repository
thrift: update (openssl3)
https://github.com/void-linux/void-packages/pull/41967
Description:
- 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
^ permalink raw reply [flat|nested] 34+ messages in thread
end of thread, other threads:[~2023-06-28 20:52 UTC | newest]
Thread overview: 34+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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 ` [PR PATCH] [Updated] " dkwo
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
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).