Github messages for voidlinux
 help / color / mirror / Atom feed
From: ologantr <ologantr@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] libtorrent-rasterbar: update to 2.0.6.
Date: Wed, 01 Jun 2022 22:53:39 +0200	[thread overview]
Message-ID: <20220601205339.RuQhjCAssVn9OaaNzVCJPG95xvfSDDmNDsC9L561vxg@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-36926@inbox.vuxu.org>

[-- Attachment #1: Type: text/plain, Size: 1199 bytes --]

There is an updated pull request by ologantr against master on the void-packages repository

https://github.com/ologantr/void-packages libtorrent-updates
https://github.com/void-linux/void-packages/pull/36926

libtorrent-rasterbar: update to 2.0.6.
#### Testing the changes
- I tested the changes in this PR: **briefly**

#### Local build testing
- I built this PR locally for my native architecture (x86_64-glibc)

I have mainly tested qbittorrent and is working as expected so far.
The patch that I have created for libtorrent-rasterbar is essentially
a revert of upstream commit https://github.com/arvidn/libtorrent/commit/55bbcdc4321dbcd899f783763d9390ecc3a09850.
Without this revert the generated pkg-config file has a bogus 
`-l-pthread` flag that then makes linking against libtorrent-rasterbar fail.
Please test :)

Edit: Deluge 2.0.5 seems stable with libtorrent-rasterbar 2.x, but some
tests are failing. For now I've disabled them.
https://git.alpinelinux.org/aports/tree/community/deluge/APKBUILD
https://github.com/deluge-torrent/deluge/pull/312 (this seems in some way related)

A patch file from https://github.com/void-linux/void-packages/pull/36926.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-libtorrent-updates-36926.patch --]
[-- Type: text/x-diff, Size: 6390 bytes --]

From 35a0997fd59cf5d6b41f9650de5672e65f49c7ad Mon Sep 17 00:00:00 2001
From: ologantr <mrphyber@protonmail.com>
Date: Sat, 30 Apr 2022 14:36:55 +0200
Subject: [PATCH 1/4] libtorrent-rasterbar: update to 2.0.6.

---
 common/shlibs                                 |  2 +-
 .../patches/remove-pthread.patch              | 10 +++++
 srcpkgs/libtorrent-rasterbar/template         | 37 ++++++++++++++-----
 3 files changed, 39 insertions(+), 10 deletions(-)
 create mode 100644 srcpkgs/libtorrent-rasterbar/patches/remove-pthread.patch

diff --git a/common/shlibs b/common/shlibs
index 0541c2dea7e8..21fbbf2ad87c 100644
--- a/common/shlibs
+++ b/common/shlibs
@@ -2081,7 +2081,7 @@ libldns.so.3 libldns-1.7.1_4
 libopenjpeg.so.5 libopenjpeg-1.5.2_1
 liboping.so.0 liboping-1.8.0_1
 libloudmouth-1.so.0 loudmouth-1.5.3_12
-libtorrent-rasterbar.so.10 libtorrent-rasterbar-1.2.12_2
+libtorrent-rasterbar.so.2.0 libtorrent-rasterbar-2.0.6_1
 libcapstone.so.4 capstone-4.0_1
 libhavege.so.2 libhaveged-1.9.11_1
 libnih.so.1 libnih-1.0.3_1
diff --git a/srcpkgs/libtorrent-rasterbar/patches/remove-pthread.patch b/srcpkgs/libtorrent-rasterbar/patches/remove-pthread.patch
new file mode 100644
index 000000000000..a421e6069753
--- /dev/null
+++ b/srcpkgs/libtorrent-rasterbar/patches/remove-pthread.patch
@@ -0,0 +1,10 @@
+--- b/CMakeLists.txt
++++ a/CMakeLists.txt
+@@ -447,7 +447,6 @@
+ feature_option(BUILD_SHARED_LIBS "build libtorrent as a shared library" ON)
+ feature_option(static_runtime "build libtorrent with static runtime" OFF)
+ 
+-set(THREADS_PREFER_PTHREAD_FLAG ON)
+ find_public_dependency(Threads REQUIRED)
+ 
+ if(CMAKE_CXX_COMPILER_ID MATCHES Clang)
diff --git a/srcpkgs/libtorrent-rasterbar/template b/srcpkgs/libtorrent-rasterbar/template
index b1ceddc95282..06a6cc47b207 100644
--- a/srcpkgs/libtorrent-rasterbar/template
+++ b/srcpkgs/libtorrent-rasterbar/template
@@ -1,10 +1,10 @@
 # Template file for 'libtorrent-rasterbar'
 # Breaks ABI/API without changing soname, revbump all dependants
 pkgname=libtorrent-rasterbar
-version=1.2.14
-revision=2
+version=2.0.6
+revision=1
 build_style=cmake
-configure_args="-DCMAKE_CXX_STANDARD=11 -Dbuild_examples=ON -Dbuild_tools=ON
+configure_args="-DCMAKE_CXX_STANDARD=14 -Dbuild_examples=ON -Dbuild_tools=ON
  -Dpython-bindings=ON"
 hostmakedepends="pkg-config intltool libtool python3-devel"
 makedepends="openssl-devel boost-devel geoip-devel python3-devel"
@@ -13,23 +13,42 @@ maintainer="Orphaned <orphan@voidlinux.org>"
 license="BSD-3-Clause"
 homepage="https://libtorrent.org/"
 distfiles="https://github.com/arvidn/libtorrent/releases/download/v${version}/${pkgname}-${version}.tar.gz"
-checksum=9e27bf359b45236d4490960faffc796528f3adbdea6aeb6881d39f310e27953f
-
-if [ "$XBPS_CHECK_PKGS" ]; then
-	configure_args+=" -Dbuild_tests=ON"
-fi
+checksum=438e29272ff41ccc68ec7530f1b98d639f6d01ec8bf680766336ae202a065722
 
 if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
 	makedepends+=" libatomic-devel"
 fi
 
+case "$XBPS_TARGET_MACHINE" in
+	i686*) make_check=no # tests fail to compile on 32bit
+	;;
+	*) [ "$XBPS_CHECK_PKGS" ] && configure_args+=" -Dbuild_tests=ON"
+	;;
+esac
+
+do_check() {
+	# taken from alpine
+	local tests_to_skip="test_upnp|test_flags|test_remove_torrent|test_privacy"
+
+	# broken, for now
+	tests_to_skip="$tests_to_skip|test_create_torrent"
+
+	# test_copy_file fails on x86_64* CI
+	case "$XBPS_TARGET_MACHINE" in
+		x86_64*) tests_to_skip="$tests_to_skip|test_url_seed|test_session_params|test_copy_file" ;;
+		*) ;;
+	esac
+
+	export CTEST_PARALLEL_LEVEL="$XBPS_MAKEJOBS"
+	ctest --output-on-failure --test-dir build --exclude-regex "$tests_to_skip"
+}
+
 pre_configure() {
 	if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
 		vsed -i CMakeLists.txt -e "s;Threads::Threads;& atomic;"
 	fi
 }
 
-
 post_install() {
 	local f
 

From 13219ddf885b9c362be967f20d905578ec95aff0 Mon Sep 17 00:00:00 2001
From: ologantr <mrphyber@protonmail.com>
Date: Sat, 30 Apr 2022 18:44:48 +0200
Subject: [PATCH 2/4] btfs: revbump for libtorrent-rasterbar.

---
 srcpkgs/btfs/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/btfs/template b/srcpkgs/btfs/template
index a41afd1e2db5..eead540d33a2 100644
--- a/srcpkgs/btfs/template
+++ b/srcpkgs/btfs/template
@@ -1,7 +1,7 @@
 # Template file for 'btfs'
 pkgname=btfs
 version=2.24
-revision=1
+revision=2
 build_style=gnu-configure
 hostmakedepends="automake pkg-config"
 makedepends="boost-devel fuse-devel libcurl-devel libtorrent-rasterbar-devel"

From 90eee77d1f13d18c55dfb11f28d476d92993b56a Mon Sep 17 00:00:00 2001
From: ologantr <mrphyber@protonmail.com>
Date: Sat, 30 Apr 2022 18:54:04 +0200
Subject: [PATCH 3/4] qbittorrent: revbump for libtorrent-rasterbar.

---
 srcpkgs/qbittorrent/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/qbittorrent/template b/srcpkgs/qbittorrent/template
index ada2ed429f26..efc0270bb61b 100644
--- a/srcpkgs/qbittorrent/template
+++ b/srcpkgs/qbittorrent/template
@@ -1,7 +1,7 @@
 # Template file for 'qbittorrent'
 pkgname=qbittorrent
 version=4.4.3.1
-revision=1
+revision=2
 create_wrksrc=yes
 build_style=gnu-configure
 build_helper=qmake

From 84e251cb21c9a26d757017ec67fbafc7114b913d Mon Sep 17 00:00:00 2001
From: ologantr <mrphyber@protonmail.com>
Date: Sat, 30 Apr 2022 18:55:18 +0200
Subject: [PATCH 4/4] deluge: revbump for libtorrent-rasterbar.

Also, disable checks for now as some of
them fail with libtorrent-rasterbar 2.x.
---
 srcpkgs/deluge/template | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/deluge/template b/srcpkgs/deluge/template
index bdb186148072..b5fa2752a6f6 100644
--- a/srcpkgs/deluge/template
+++ b/srcpkgs/deluge/template
@@ -1,7 +1,7 @@
 # Template file for 'deluge'
 pkgname=deluge
 version=2.0.5
-revision=1
+revision=2
 build_style=python3-module
 # TODO package python3-slimit to minify javascript
 hostmakedepends="intltool python3-setuptools python3-wheel"
@@ -17,6 +17,7 @@ homepage="https://deluge-torrent.org/"
 changelog="https://raw.githubusercontent.com/deluge-torrent/deluge/develop/CHANGELOG.md"
 distfiles="https://ftp.osuosl.org/pub/deluge/source/2.0/deluge-${version}.tar.xz"
 checksum=c4bd04abfd211b65218be03f3c46d26f44024884de10e01859fb856fdd6f25d8
+make_check=no # one test segfaults, another is failing, taken from alpine
 
 system_accounts="deluge"
 deluge_homedir="/var/lib/deluge"

  parent reply	other threads:[~2022-06-01 20:53 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-30 17:29 [PR PATCH] " ologantr
2022-04-30 19:55 ` [PR PATCH] [Updated] " ologantr
2022-06-01 15:20 ` ologantr
2022-06-01 20:53 ` ologantr [this message]
2022-09-29  2:14 ` github-actions
2022-10-13  2:15 ` [PR PATCH] [Closed]: " github-actions

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220601205339.RuQhjCAssVn9OaaNzVCJPG95xvfSDDmNDsC9L561vxg@z \
    --to=ologantr@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).