Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v2 1/2] package/libutp: bump version
@ 2023-05-06 16:05 Bernd Kuhls
  2023-05-06 16:05 ` [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3 Bernd Kuhls
  2023-05-07 16:18 ` [Buildroot] [PATCH v2 1/2] package/libutp: bump version Yann E. MORIN
  0 siblings, 2 replies; 7+ messages in thread
From: Bernd Kuhls @ 2023-05-06 16:05 UTC (permalink / raw)
  To: buildroot; +Cc: Samuel Martin

Needed for bumping transmission to version 4.0.1.

Added configure option to fix build error with gcc-12.

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: No changes

 package/libutp/libutp.hash | 2 +-
 package/libutp/libutp.mk   | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/package/libutp/libutp.hash b/package/libutp/libutp.hash
index f347dc3f2c..0312f42612 100644
--- a/package/libutp/libutp.hash
+++ b/package/libutp/libutp.hash
@@ -1,3 +1,3 @@
 # Locally computed
-sha256  484f802d044f774fa5f669476ecdd9cb6110c383781c39adc03bbf6575090771  libutp-059c9449a104e440e4f913756a5f560dd4ae76a9.tar.gz
+sha256  d856fde68828d52eb39df40e15ad5dc4efaa9a51d4121bcbfbe47fed2163d20a  libutp-c95738b1a6644b919e5b64d3ea9736cfc5894e0b.tar.gz
 sha256  73b400bf61dde52223dae1642205ca187f1a6b24a508dcce52bf386c27a0e58c  LICENSE
diff --git a/package/libutp/libutp.mk b/package/libutp/libutp.mk
index 394c1e9cb9..df0617555d 100644
--- a/package/libutp/libutp.mk
+++ b/package/libutp/libutp.mk
@@ -4,10 +4,11 @@
 #
 ################################################################################
 
-LIBUTP_VERSION = 059c9449a104e440e4f913756a5f560dd4ae76a9
+LIBUTP_VERSION = c95738b1a6644b919e5b64d3ea9736cfc5894e0b
 LIBUTP_SITE = $(call github,transmission,libutp,$(LIBUTP_VERSION))
 LIBUTP_LICENSE = MIT
 LIBUTP_LICENSE_FILES = LICENSE
 LIBUTP_INSTALL_STAGING = YES
+LIBUTP_CONF_OPTS = -DLIBUTP_ENABLE_WERROR=OFF
 
 $(eval $(cmake-package))
-- 
2.39.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3
  2023-05-06 16:05 [Buildroot] [PATCH v2 1/2] package/libutp: bump version Bernd Kuhls
@ 2023-05-06 16:05 ` Bernd Kuhls
  2023-05-07 10:52   ` Yann E. MORIN
                     ` (2 more replies)
  2023-05-07 16:18 ` [Buildroot] [PATCH v2 1/2] package/libutp: bump version Yann E. MORIN
  1 sibling, 3 replies; 7+ messages in thread
From: Bernd Kuhls @ 2023-05-06 16:05 UTC (permalink / raw)
  To: buildroot; +Cc: Samuel Martin

This bump depends on bumping libutp to at least
c95738b1a6644b919e5b64d3ea9736cfc5894e0b.

Updated license hash due to copyright year bump:
https://github.com/transmission/transmission/commit/4b8cfa2e576feea6a6f2fd5ae5fe1a59449aca6f

Switched buildsystem to cmake, reworked dependencies.

Removed all patches, transmission now depends on the libutp package and
uses cmake as build system.

Upstream removed the configure option for lightweight build:
https://github.com/transmission/transmission/commit/47ebb3f63a2f8983f9858bfbec6b72c42588a388

Removed the option to disable uTP support, for details see upstream issue:
https://github.com/transmission/transmission/issues/4751

Added -latomic when needed to fix build errors on riscv32 and sparc.

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: bump version to 4.0.3

 package/transmission/0001-fix-utypes.patch    | 30 ----------
 .../transmission/0002-fix-utp-include.patch   | 46 ---------------
 ...ac-fix-autoreconf-with-autoconf-2.70.patch | 42 -------------
 package/transmission/Config.in                | 32 +++++-----
 package/transmission/transmission.hash        |  5 +-
 package/transmission/transmission.mk          | 59 +++++++++----------
 6 files changed, 50 insertions(+), 164 deletions(-)
 delete mode 100644 package/transmission/0001-fix-utypes.patch
 delete mode 100644 package/transmission/0002-fix-utp-include.patch
 delete mode 100644 package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch

diff --git a/package/transmission/0001-fix-utypes.patch b/package/transmission/0001-fix-utypes.patch
deleted file mode 100644
index 81efa884a7..0000000000
--- a/package/transmission/0001-fix-utypes.patch
+++ /dev/null
@@ -1,30 +0,0 @@
-Fix build error "conflicting types for 'UTP_Write'"
-
-Fetch from: http://www.adminsehow.com/2012/10/how-to-install-transmission-2-71-on-debian-6-0-squeeze/
-
-The corresponding upstream bug report (unfixed) can be found here:
-https://trac.transmissionbt.com/ticket/5232
-
-Signed-off-by: Bernd Kuhls <berndkuhls@hotmail.com>
-[Fabrice: adapt to 3.00, which always uses stdbool.h]
-Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-[yann.morin.1998@free.fr:
-  - don't comment-out dropped code, just drop it
-  - use CR-LF in replacement code, as used by upstream
-]
-Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
-
---- transmission-2.82.org/third-party/libutp/utypes.h	2013-08-09 04:47:43.000000000 +0200
-+++ transmission-2.82/third-party/libutp/utypes.h	2014-01-02 20:17:18.000000000 +0100
-@@ -35,8 +35,8 @@
- typedef const char * cstr;
- typedef char * str;
- 
--#ifndef __cplusplus
--typedef uint8 bool;
--#endif
-+#ifndef __cplusplus
-+#include <stdbool.h>
-+#endif
- 
- #endif //__UTYPES_H__
diff --git a/package/transmission/0002-fix-utp-include.patch b/package/transmission/0002-fix-utp-include.patch
deleted file mode 100644
index db8b5ab39b..0000000000
--- a/package/transmission/0002-fix-utp-include.patch
+++ /dev/null
@@ -1,46 +0,0 @@
-Fix compilation when using system libminiupnpc with libutp disabled
-
-If transmission uses its own copy of libminiupnpc in third-party/ then
-the configure script adds that path to CFLAGS:
-https://trac.transmissionbt.com/browser/trunk/configure.ac#L455
-
-This path is also added if transmission uses its own copy of libutp:
-https://trac.transmissionbt.com/browser/trunk/configure.ac#L349
-
-But if a system-provided libminiupnpc is used and utp-support is
-disabled using --disable-utp the result is a build error, detected by
-buildroot autobuilders:
-http://autobuild.buildroot.net/results/b79/b79e51140cb0320554a58ce59dcb33f53cd9211a/build-end.log
-
-net.c:40:24: fatal error: libutp/utp.h: No such file or directory
-
-Patch proposed by Yann E. MORIN:
-http://lists.busybox.net/pipermail/buildroot/2016-July/167896.html
-
-Patch sent upstream: https://trac.transmissionbt.com/ticket/6163
-
-Signed-off-by: Bernd Kuhls <berndkuhls@hotmail.com>
-
-diff -uNr transmission-2.92.org/configure.ac transmission-2.92/configure.ac
---- transmission-2.92.org/configure.ac	2016-03-06 21:23:45.336198393 +0100
-+++ transmission-2.92/configure.ac	2016-07-31 12:13:40.921660078 +0200
-@@ -358,6 +358,10 @@
-     else
-       AC_MSG_ERROR("Unable to build uTP support -- C++ compiler not found")
-     fi
-+else
-+    # For the system libminiupnpc with UTC disabled,
-+    # point to our bundled UTP headers
-+    LIBUTP_CFLAGS="-I\$(top_srcdir)/third-party/"
- fi
- AC_SUBST(LIBUTP_CFLAGS)
- AC_SUBST(LIBUTP_LIBS)
-@@ -457,7 +461,7 @@
-     LIBUPNP_LIBS_QT="\$\${TRANSMISSION_TOP}/third-party/miniupnp/libminiupnp.a"
- else
-     AC_DEFINE([SYSTEM_MINIUPNP])
--    LIBUPNP_CFLAGS="" 
-+    LIBUPNP_CFLAGS="${LIBUTP_CFLAGS}"
-     LIBUPNP_LIBS="-lminiupnpc"
-     LIBUPNP_LIBS_QT="-lminiupnpc"
- fi
diff --git a/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch b/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch
deleted file mode 100644
index bfb0084bc4..0000000000
--- a/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch
+++ /dev/null
@@ -1,42 +0,0 @@
-From 857ac91a03a096601f62b358fc61355d4cc25b9b Mon Sep 17 00:00:00 2001
-From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-Date: Sat, 9 Oct 2021 18:51:13 +0200
-Subject: [PATCH] configure.ac: fix autoreconf with autoconf >= 2.70
-
-Remove the whitespace before IT_PROG_INTLTOOL as suggested in
-https://www.mail-archive.com/bug-autoconf@gnu.org/msg04851.html
-to avoid the following build failure with autoconf >= 2.70:
-
-ERROR: 'IT_PROG_INTLTOOL' must appear in configure.ac for intltool to work.
-
-The problem has been introduced by upstream autoconf commit
-http://git.savannah.gnu.org/gitweb/?p=autoconf.git;a=commit;h=b3b3af821a0b9e82ebe56b77b1731d2fe58f52de
-
-Fixes:
- - http://autobuild.buildroot.org/results/e8bea26f6b12adf16335b24836a7c5c31911af6a
-
-Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-[Upstream status: not upstreamable as upstream swicthed to cmake (
-https://github.com/transmission/transmission/issues/1573)]
----
- configure.ac | 4 +---
- 1 file changed, 1 insertion(+), 3 deletions(-)
-
-diff --git a/configure.ac b/configure.ac
-index fd1132fda..04e0e19bb 100644
---- a/configure.ac
-+++ b/configure.ac
-@@ -555,9 +555,7 @@ dnl it should be safe to re-edit 0.40 back down to 0.23
- use_nls=no
- if test "x$enable_nls" = "xyes" ; then
-     use_nls=yes
--    m4_ifdef([IT_PROG_INTLTOOL],
--             [IT_PROG_INTLTOOL([0.35.0],[no-xml])],
--             [AC_MSG_ERROR("--enable-nls requires intltool to be installed.")])
-+IT_PROG_INTLTOOL([0.35.0],[no-xml])
-     AC_CHECK_HEADERS([libintl.h])
-     GETTEXT_PACKAGE=transmission-gtk
-     AC_SUBST(GETTEXT_PACKAGE)
--- 
-2.33.0
-
diff --git a/package/transmission/Config.in b/package/transmission/Config.in
index 980b042e33..84bd85eeff 100644
--- a/package/transmission/Config.in
+++ b/package/transmission/Config.in
@@ -1,12 +1,23 @@
 config BR2_PACKAGE_TRANSMISSION
 	bool "transmission"
+	depends on !BR2_BINFMT_FLAT # libnatpmp
+	depends on BR2_INSTALL_LIBSTDCPP # libutp
+	depends on !BR2_STATIC_LIBS # libnatpmp
+	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_7 # C++17
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_USE_MMU # fork()
-	select BR2_PACKAGE_ZLIB
-	select BR2_PACKAGE_OPENSSL
-	select BR2_PACKAGE_LIBOPENSSL_ENABLE_RC4 if BR2_PACKAGE_LIBOPENSSL
+	depends on BR2_USE_WCHAR # libpsl
+	select BR2_PACKAGE_DHT
+	select BR2_PACKAGE_LIBB64
 	select BR2_PACKAGE_LIBCURL
+	select BR2_PACKAGE_LIBDEFLATE
 	select BR2_PACKAGE_LIBEVENT
+	select BR2_PACKAGE_LIBMINIUPNPC
+	select BR2_PACKAGE_LIBNATPMP
+	select BR2_PACKAGE_LIBPSL
+	select BR2_PACKAGE_LIBUTP
+	select BR2_PACKAGE_OPENSSL
+	select BR2_PACKAGE_ZLIB
 	help
 	  Transmission is a cross-platform BitTorrent client.
 
@@ -14,15 +25,6 @@ config BR2_PACKAGE_TRANSMISSION
 
 if BR2_PACKAGE_TRANSMISSION
 
-config BR2_PACKAGE_TRANSMISSION_UTP
-	bool "Transmission uTP support"
-	depends on BR2_INSTALL_LIBSTDCPP
-	help
-	  Install support for uTorrent Transport Protocol
-
-comment "uTP needs a toolchain w/ C++"
-	depends on !BR2_INSTALL_LIBSTDCPP
-
 config BR2_PACKAGE_TRANSMISSION_CLI
 	bool "transmission-cli"
 	help
@@ -43,6 +45,8 @@ comment "transmission-gtk needs NLS enabled"
 	depends on BR2_PACKAGE_LIBGTK3 && !BR2_SYSTEM_ENABLE_NLS
 endif
 
-comment "transmission needs a toolchain w/ threads"
+comment "transmission needs a toolchain w/ dynamic library, threads, C++, gcc >= 7"
 	depends on BR2_USE_MMU
-	depends on !BR2_TOOLCHAIN_HAS_THREADS
+	depends on !BR2_BINFMT_FLAT
+	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_INSTALL_LIBSTDCPP || \
+		!BR2_TOOLCHAIN_GCC_AT_LEAST_7 || !BR2_STATIC_LIBS
diff --git a/package/transmission/transmission.hash b/package/transmission/transmission.hash
index 13cbc969d3..e6775c47e7 100644
--- a/package/transmission/transmission.hash
+++ b/package/transmission/transmission.hash
@@ -1,3 +1,4 @@
+# From https://transmissionbt.com/download
+sha256  b6b01fd58e42bb14f7aba0253db932ced050fcd2bba5d9f8469d77ddd8ad545a  transmission-4.0.3.tar.xz
 # Locally calculated
-sha256  9144652fe742f7f7dd6657716e378da60b751aaeda8bef8344b3eefc4db255f2  transmission-3.00.tar.xz
-sha256  2b16da5bd02790d1e4d332cfd42b9dfd7d1b0ed3a16a75dc7e76b6ed239fdba9  COPYING
+sha256  f56732960a61ecf3b9637404eef1a39221d2006336a98792b7b65a79f155449c  COPYING
diff --git a/package/transmission/transmission.mk b/package/transmission/transmission.mk
index 1660d2c2b9..f3a200ced3 100644
--- a/package/transmission/transmission.mk
+++ b/package/transmission/transmission.mk
@@ -4,55 +4,54 @@
 #
 ################################################################################
 
-TRANSMISSION_VERSION = 3.00
-TRANSMISSION_SITE = https://github.com/transmission/transmission-releases/raw/master
+TRANSMISSION_VERSION = 4.0.3
 TRANSMISSION_SOURCE = transmission-$(TRANSMISSION_VERSION).tar.xz
+TRANSMISSION_SITE = https://github.com/transmission/transmission/releases/download/$(TRANSMISSION_VERSION)
 TRANSMISSION_DEPENDENCIES = \
 	host-pkgconf \
-	host-intltool \
+	dht \
+	libb64 \
 	libcurl \
+	libdeflate \
 	libevent \
+	libminiupnpc \
+	libnatpmp \
+	libpsl \
+	libutp \
 	openssl \
 	zlib
-TRANSMISSION_AUTORECONF = YES
 TRANSMISSION_CONF_OPTS = \
-	--without-inotify \
-	--enable-lightweight
+	-DENABLE_TESTS=OFF \
+	-DRUN_CLANG_TIDY=OFF \
+	-DUSE_SYSTEM_B64=ON \
+	-DUSE_SYSTEM_DEFLATE=ON \
+	-DUSE_SYSTEM_DHT=ON \
+	-DUSE_SYSTEM_NATPMP=ON \
+	-DUSE_SYSTEM_PSL=ON \
+	-DWITH_INOTIFY=OFF
 TRANSMISSION_LICENSE = GPL-2.0 or GPL-3.0 with OpenSSL exception
 TRANSMISSION_LICENSE_FILES = COPYING
 TRANSMISSION_CPE_ID_VENDOR = transmissionbt
 
-ifeq ($(BR2_PACKAGE_LIBMINIUPNPC),y)
-TRANSMISSION_DEPENDENCIES += libminiupnpc
-endif
-
-ifeq ($(BR2_PACKAGE_LIBNATPMP),y)
-TRANSMISSION_DEPENDENCIES += libnatpmp
-TRANSMISSION_CONF_OPTS += --enable-external-natpmp
-else
-TRANSMISSION_CONF_OPTS += --disable-external-natpmp
-endif
-
-ifeq ($(BR2_PACKAGE_TRANSMISSION_UTP),y)
-TRANSMISSION_CONF_OPTS += --enable-utp
-else
-TRANSMISSION_CONF_OPTS += --disable-utp
+# Uses __atomic_load_8
+ifeq ($(BR2_TOOLCHAIN_HAS_LIBATOMIC),y)
+TRANSMISSION_CONF_OPTS += -DCMAKE_EXE_LINKER_FLAGS=-latomic
 endif
 
 ifeq ($(BR2_PACKAGE_TRANSMISSION_CLI),y)
-TRANSMISSION_CONF_OPTS += --enable-cli
+TRANSMISSION_CONF_OPTS += -DENABLE_CLI=ON
 else
-TRANSMISSION_CONF_OPTS += --disable-cli
+TRANSMISSION_CONF_OPTS += -DENABLE_CLI=OFF
 endif
 
 ifeq ($(BR2_PACKAGE_TRANSMISSION_DAEMON),y)
-TRANSMISSION_CONF_OPTS += --enable-daemon
+TRANSMISSION_CONF_OPTS += -DENABLE_DAEMON=ON
 
 ifeq ($(BR2_PACKAGE_SYSTEMD),y)
 TRANSMISSION_DEPENDENCIES += systemd
-TRANSMISSION_CONF_OPTS += --with-systemd
+TRANSMISSION_CONF_OPTS += -DWITH_SYSTEMD=ON
 else
-TRANSMISSION_CONF_OPTS += --without-systemd
+TRANSMISSION_CONF_OPTS += -DWITH_SYSTEMD=OFF
 endif
 
 define TRANSMISSION_USERS
@@ -70,14 +69,14 @@ define TRANSMISSION_INSTALL_INIT_SYSTEMD
 endef
 
 else
-TRANSMISSION_CONF_OPTS += --disable-daemon
+TRANSMISSION_CONF_OPTS += -DENABLE_DAEMON=OFF
 endif
 
 ifeq ($(BR2_PACKAGE_TRANSMISSION_GTK),y)
-TRANSMISSION_CONF_OPTS += --with-gtk
+TRANSMISSION_CONF_OPTS += -DENABLE_GTK=ON
 TRANSMISSION_DEPENDENCIES += libgtk3
 else
-TRANSMISSION_CONF_OPTS += --without-gtk
+TRANSMISSION_CONF_OPTS += -DENABLE_GTK=OFF
 endif
 
-$(eval $(autotools-package))
+$(eval $(cmake-package))
-- 
2.39.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3
  2023-05-06 16:05 ` [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3 Bernd Kuhls
@ 2023-05-07 10:52   ` Yann E. MORIN
       [not found]   ` <20230507105218.GH252090__6425.69195931486$1683456775$gmane$org@scaer>
  2023-05-07 16:23   ` Yann E. MORIN
  2 siblings, 0 replies; 7+ messages in thread
From: Yann E. MORIN @ 2023-05-07 10:52 UTC (permalink / raw)
  To: Bernd Kuhls; +Cc: Samuel Martin, buildroot

Bernd, All,

On 2023-05-06 18:05 +0200, Bernd Kuhls spake thusly:
> This bump depends on bumping libutp to at least
> c95738b1a6644b919e5b64d3ea9736cfc5894e0b.
> 
> Updated license hash due to copyright year bump:
> https://github.com/transmission/transmission/commit/4b8cfa2e576feea6a6f2fd5ae5fe1a59449aca6f
> 
> Switched buildsystem to cmake, reworked dependencies.
> 
> Removed all patches, transmission now depends on the libutp package and
> uses cmake as build system.
> 
> Upstream removed the configure option for lightweight build:
> https://github.com/transmission/transmission/commit/47ebb3f63a2f8983f9858bfbec6b72c42588a388
> 
> Removed the option to disable uTP support, for details see upstream issue:
> https://github.com/transmission/transmission/issues/4751

The option is still present, so why would we want to remove it?

    https://github.com/transmission/transmission/blob/main/CMakeLists.txt
    57  option(ENABLE_UTP "Build µTP support" ON)

Regards,
Yann E. MORIN.

> Added -latomic when needed to fix build errors on riscv32 and sparc.
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v2: bump version to 4.0.3
> 
>  package/transmission/0001-fix-utypes.patch    | 30 ----------
>  .../transmission/0002-fix-utp-include.patch   | 46 ---------------
>  ...ac-fix-autoreconf-with-autoconf-2.70.patch | 42 -------------
>  package/transmission/Config.in                | 32 +++++-----
>  package/transmission/transmission.hash        |  5 +-
>  package/transmission/transmission.mk          | 59 +++++++++----------
>  6 files changed, 50 insertions(+), 164 deletions(-)
>  delete mode 100644 package/transmission/0001-fix-utypes.patch
>  delete mode 100644 package/transmission/0002-fix-utp-include.patch
>  delete mode 100644 package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch
> 
> diff --git a/package/transmission/0001-fix-utypes.patch b/package/transmission/0001-fix-utypes.patch
> deleted file mode 100644
> index 81efa884a7..0000000000
> --- a/package/transmission/0001-fix-utypes.patch
> +++ /dev/null
> @@ -1,30 +0,0 @@
> -Fix build error "conflicting types for 'UTP_Write'"
> -
> -Fetch from: http://www.adminsehow.com/2012/10/how-to-install-transmission-2-71-on-debian-6-0-squeeze/
> -
> -The corresponding upstream bug report (unfixed) can be found here:
> -https://trac.transmissionbt.com/ticket/5232
> -
> -Signed-off-by: Bernd Kuhls <berndkuhls@hotmail.com>
> -[Fabrice: adapt to 3.00, which always uses stdbool.h]
> -Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -[yann.morin.1998@free.fr:
> -  - don't comment-out dropped code, just drop it
> -  - use CR-LF in replacement code, as used by upstream
> -]
> -Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> -
> ---- transmission-2.82.org/third-party/libutp/utypes.h	2013-08-09 04:47:43.000000000 +0200
> -+++ transmission-2.82/third-party/libutp/utypes.h	2014-01-02 20:17:18.000000000 +0100
> -@@ -35,8 +35,8 @@
> - typedef const char * cstr;
> - typedef char * str;
> - 
> --#ifndef __cplusplus
> --typedef uint8 bool;
> --#endif
> -+#ifndef __cplusplus
> -+#include <stdbool.h>
> -+#endif
> - 
> - #endif //__UTYPES_H__
> diff --git a/package/transmission/0002-fix-utp-include.patch b/package/transmission/0002-fix-utp-include.patch
> deleted file mode 100644
> index db8b5ab39b..0000000000
> --- a/package/transmission/0002-fix-utp-include.patch
> +++ /dev/null
> @@ -1,46 +0,0 @@
> -Fix compilation when using system libminiupnpc with libutp disabled
> -
> -If transmission uses its own copy of libminiupnpc in third-party/ then
> -the configure script adds that path to CFLAGS:
> -https://trac.transmissionbt.com/browser/trunk/configure.ac#L455
> -
> -This path is also added if transmission uses its own copy of libutp:
> -https://trac.transmissionbt.com/browser/trunk/configure.ac#L349
> -
> -But if a system-provided libminiupnpc is used and utp-support is
> -disabled using --disable-utp the result is a build error, detected by
> -buildroot autobuilders:
> -http://autobuild.buildroot.net/results/b79/b79e51140cb0320554a58ce59dcb33f53cd9211a/build-end.log
> -
> -net.c:40:24: fatal error: libutp/utp.h: No such file or directory
> -
> -Patch proposed by Yann E. MORIN:
> -http://lists.busybox.net/pipermail/buildroot/2016-July/167896.html
> -
> -Patch sent upstream: https://trac.transmissionbt.com/ticket/6163
> -
> -Signed-off-by: Bernd Kuhls <berndkuhls@hotmail.com>
> -
> -diff -uNr transmission-2.92.org/configure.ac transmission-2.92/configure.ac
> ---- transmission-2.92.org/configure.ac	2016-03-06 21:23:45.336198393 +0100
> -+++ transmission-2.92/configure.ac	2016-07-31 12:13:40.921660078 +0200
> -@@ -358,6 +358,10 @@
> -     else
> -       AC_MSG_ERROR("Unable to build uTP support -- C++ compiler not found")
> -     fi
> -+else
> -+    # For the system libminiupnpc with UTC disabled,
> -+    # point to our bundled UTP headers
> -+    LIBUTP_CFLAGS="-I\$(top_srcdir)/third-party/"
> - fi
> - AC_SUBST(LIBUTP_CFLAGS)
> - AC_SUBST(LIBUTP_LIBS)
> -@@ -457,7 +461,7 @@
> -     LIBUPNP_LIBS_QT="\$\${TRANSMISSION_TOP}/third-party/miniupnp/libminiupnp.a"
> - else
> -     AC_DEFINE([SYSTEM_MINIUPNP])
> --    LIBUPNP_CFLAGS="" 
> -+    LIBUPNP_CFLAGS="${LIBUTP_CFLAGS}"
> -     LIBUPNP_LIBS="-lminiupnpc"
> -     LIBUPNP_LIBS_QT="-lminiupnpc"
> - fi
> diff --git a/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch b/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch
> deleted file mode 100644
> index bfb0084bc4..0000000000
> --- a/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch
> +++ /dev/null
> @@ -1,42 +0,0 @@
> -From 857ac91a03a096601f62b358fc61355d4cc25b9b Mon Sep 17 00:00:00 2001
> -From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -Date: Sat, 9 Oct 2021 18:51:13 +0200
> -Subject: [PATCH] configure.ac: fix autoreconf with autoconf >= 2.70
> -
> -Remove the whitespace before IT_PROG_INTLTOOL as suggested in
> -https://www.mail-archive.com/bug-autoconf@gnu.org/msg04851.html
> -to avoid the following build failure with autoconf >= 2.70:
> -
> -ERROR: 'IT_PROG_INTLTOOL' must appear in configure.ac for intltool to work.
> -
> -The problem has been introduced by upstream autoconf commit
> -http://git.savannah.gnu.org/gitweb/?p=autoconf.git;a=commit;h=b3b3af821a0b9e82ebe56b77b1731d2fe58f52de
> -
> -Fixes:
> - - http://autobuild.buildroot.org/results/e8bea26f6b12adf16335b24836a7c5c31911af6a
> -
> -Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -[Upstream status: not upstreamable as upstream swicthed to cmake (
> -https://github.com/transmission/transmission/issues/1573)]
> ----
> - configure.ac | 4 +---
> - 1 file changed, 1 insertion(+), 3 deletions(-)
> -
> -diff --git a/configure.ac b/configure.ac
> -index fd1132fda..04e0e19bb 100644
> ---- a/configure.ac
> -+++ b/configure.ac
> -@@ -555,9 +555,7 @@ dnl it should be safe to re-edit 0.40 back down to 0.23
> - use_nls=no
> - if test "x$enable_nls" = "xyes" ; then
> -     use_nls=yes
> --    m4_ifdef([IT_PROG_INTLTOOL],
> --             [IT_PROG_INTLTOOL([0.35.0],[no-xml])],
> --             [AC_MSG_ERROR("--enable-nls requires intltool to be installed.")])
> -+IT_PROG_INTLTOOL([0.35.0],[no-xml])
> -     AC_CHECK_HEADERS([libintl.h])
> -     GETTEXT_PACKAGE=transmission-gtk
> -     AC_SUBST(GETTEXT_PACKAGE)
> --- 
> -2.33.0
> -
> diff --git a/package/transmission/Config.in b/package/transmission/Config.in
> index 980b042e33..84bd85eeff 100644
> --- a/package/transmission/Config.in
> +++ b/package/transmission/Config.in
> @@ -1,12 +1,23 @@
>  config BR2_PACKAGE_TRANSMISSION
>  	bool "transmission"
> +	depends on !BR2_BINFMT_FLAT # libnatpmp
> +	depends on BR2_INSTALL_LIBSTDCPP # libutp
> +	depends on !BR2_STATIC_LIBS # libnatpmp
> +	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_7 # C++17
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
>  	depends on BR2_USE_MMU # fork()
> -	select BR2_PACKAGE_ZLIB
> -	select BR2_PACKAGE_OPENSSL
> -	select BR2_PACKAGE_LIBOPENSSL_ENABLE_RC4 if BR2_PACKAGE_LIBOPENSSL
> +	depends on BR2_USE_WCHAR # libpsl
> +	select BR2_PACKAGE_DHT
> +	select BR2_PACKAGE_LIBB64
>  	select BR2_PACKAGE_LIBCURL
> +	select BR2_PACKAGE_LIBDEFLATE
>  	select BR2_PACKAGE_LIBEVENT
> +	select BR2_PACKAGE_LIBMINIUPNPC
> +	select BR2_PACKAGE_LIBNATPMP
> +	select BR2_PACKAGE_LIBPSL
> +	select BR2_PACKAGE_LIBUTP
> +	select BR2_PACKAGE_OPENSSL
> +	select BR2_PACKAGE_ZLIB
>  	help
>  	  Transmission is a cross-platform BitTorrent client.
>  
> @@ -14,15 +25,6 @@ config BR2_PACKAGE_TRANSMISSION
>  
>  if BR2_PACKAGE_TRANSMISSION
>  
> -config BR2_PACKAGE_TRANSMISSION_UTP
> -	bool "Transmission uTP support"
> -	depends on BR2_INSTALL_LIBSTDCPP
> -	help
> -	  Install support for uTorrent Transport Protocol
> -
> -comment "uTP needs a toolchain w/ C++"
> -	depends on !BR2_INSTALL_LIBSTDCPP
> -
>  config BR2_PACKAGE_TRANSMISSION_CLI
>  	bool "transmission-cli"
>  	help
> @@ -43,6 +45,8 @@ comment "transmission-gtk needs NLS enabled"
>  	depends on BR2_PACKAGE_LIBGTK3 && !BR2_SYSTEM_ENABLE_NLS
>  endif
>  
> -comment "transmission needs a toolchain w/ threads"
> +comment "transmission needs a toolchain w/ dynamic library, threads, C++, gcc >= 7"
>  	depends on BR2_USE_MMU
> -	depends on !BR2_TOOLCHAIN_HAS_THREADS
> +	depends on !BR2_BINFMT_FLAT
> +	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_INSTALL_LIBSTDCPP || \
> +		!BR2_TOOLCHAIN_GCC_AT_LEAST_7 || !BR2_STATIC_LIBS
> diff --git a/package/transmission/transmission.hash b/package/transmission/transmission.hash
> index 13cbc969d3..e6775c47e7 100644
> --- a/package/transmission/transmission.hash
> +++ b/package/transmission/transmission.hash
> @@ -1,3 +1,4 @@
> +# From https://transmissionbt.com/download
> +sha256  b6b01fd58e42bb14f7aba0253db932ced050fcd2bba5d9f8469d77ddd8ad545a  transmission-4.0.3.tar.xz
>  # Locally calculated
> -sha256  9144652fe742f7f7dd6657716e378da60b751aaeda8bef8344b3eefc4db255f2  transmission-3.00.tar.xz
> -sha256  2b16da5bd02790d1e4d332cfd42b9dfd7d1b0ed3a16a75dc7e76b6ed239fdba9  COPYING
> +sha256  f56732960a61ecf3b9637404eef1a39221d2006336a98792b7b65a79f155449c  COPYING
> diff --git a/package/transmission/transmission.mk b/package/transmission/transmission.mk
> index 1660d2c2b9..f3a200ced3 100644
> --- a/package/transmission/transmission.mk
> +++ b/package/transmission/transmission.mk
> @@ -4,55 +4,54 @@
>  #
>  ################################################################################
>  
> -TRANSMISSION_VERSION = 3.00
> -TRANSMISSION_SITE = https://github.com/transmission/transmission-releases/raw/master
> +TRANSMISSION_VERSION = 4.0.3
>  TRANSMISSION_SOURCE = transmission-$(TRANSMISSION_VERSION).tar.xz
> +TRANSMISSION_SITE = https://github.com/transmission/transmission/releases/download/$(TRANSMISSION_VERSION)
>  TRANSMISSION_DEPENDENCIES = \
>  	host-pkgconf \
> -	host-intltool \
> +	dht \
> +	libb64 \
>  	libcurl \
> +	libdeflate \
>  	libevent \
> +	libminiupnpc \
> +	libnatpmp \
> +	libpsl \
> +	libutp \
>  	openssl \
>  	zlib
> -TRANSMISSION_AUTORECONF = YES
>  TRANSMISSION_CONF_OPTS = \
> -	--without-inotify \
> -	--enable-lightweight
> +	-DENABLE_TESTS=OFF \
> +	-DRUN_CLANG_TIDY=OFF \
> +	-DUSE_SYSTEM_B64=ON \
> +	-DUSE_SYSTEM_DEFLATE=ON \
> +	-DUSE_SYSTEM_DHT=ON \
> +	-DUSE_SYSTEM_NATPMP=ON \
> +	-DUSE_SYSTEM_PSL=ON \
> +	-DWITH_INOTIFY=OFF
>  TRANSMISSION_LICENSE = GPL-2.0 or GPL-3.0 with OpenSSL exception
>  TRANSMISSION_LICENSE_FILES = COPYING
>  TRANSMISSION_CPE_ID_VENDOR = transmissionbt
>  
> -ifeq ($(BR2_PACKAGE_LIBMINIUPNPC),y)
> -TRANSMISSION_DEPENDENCIES += libminiupnpc
> -endif
> -
> -ifeq ($(BR2_PACKAGE_LIBNATPMP),y)
> -TRANSMISSION_DEPENDENCIES += libnatpmp
> -TRANSMISSION_CONF_OPTS += --enable-external-natpmp
> -else
> -TRANSMISSION_CONF_OPTS += --disable-external-natpmp
> -endif
> -
> -ifeq ($(BR2_PACKAGE_TRANSMISSION_UTP),y)
> -TRANSMISSION_CONF_OPTS += --enable-utp
> -else
> -TRANSMISSION_CONF_OPTS += --disable-utp
> +# Uses __atomic_load_8
> +ifeq ($(BR2_TOOLCHAIN_HAS_LIBATOMIC),y)
> +TRANSMISSION_CONF_OPTS += -DCMAKE_EXE_LINKER_FLAGS=-latomic
>  endif
>  
>  ifeq ($(BR2_PACKAGE_TRANSMISSION_CLI),y)
> -TRANSMISSION_CONF_OPTS += --enable-cli
> +TRANSMISSION_CONF_OPTS += -DENABLE_CLI=ON
>  else
> -TRANSMISSION_CONF_OPTS += --disable-cli
> +TRANSMISSION_CONF_OPTS += -DENABLE_CLI=OFF
>  endif
>  
>  ifeq ($(BR2_PACKAGE_TRANSMISSION_DAEMON),y)
> -TRANSMISSION_CONF_OPTS += --enable-daemon
> +TRANSMISSION_CONF_OPTS += -DENABLE_DAEMON=ON
>  
>  ifeq ($(BR2_PACKAGE_SYSTEMD),y)
>  TRANSMISSION_DEPENDENCIES += systemd
> -TRANSMISSION_CONF_OPTS += --with-systemd
> +TRANSMISSION_CONF_OPTS += -DWITH_SYSTEMD=ON
>  else
> -TRANSMISSION_CONF_OPTS += --without-systemd
> +TRANSMISSION_CONF_OPTS += -DWITH_SYSTEMD=OFF
>  endif
>  
>  define TRANSMISSION_USERS
> @@ -70,14 +69,14 @@ define TRANSMISSION_INSTALL_INIT_SYSTEMD
>  endef
>  
>  else
> -TRANSMISSION_CONF_OPTS += --disable-daemon
> +TRANSMISSION_CONF_OPTS += -DENABLE_DAEMON=OFF
>  endif
>  
>  ifeq ($(BR2_PACKAGE_TRANSMISSION_GTK),y)
> -TRANSMISSION_CONF_OPTS += --with-gtk
> +TRANSMISSION_CONF_OPTS += -DENABLE_GTK=ON
>  TRANSMISSION_DEPENDENCIES += libgtk3
>  else
> -TRANSMISSION_CONF_OPTS += --without-gtk
> +TRANSMISSION_CONF_OPTS += -DENABLE_GTK=OFF
>  endif
>  
> -$(eval $(autotools-package))
> +$(eval $(cmake-package))
> -- 
> 2.39.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3
       [not found]   ` <20230507105218.GH252090__6425.69195931486$1683456775$gmane$org@scaer>
@ 2023-05-07 15:06     ` Bernd Kuhls
  2023-05-07 15:51       ` Yann E. MORIN
  0 siblings, 1 reply; 7+ messages in thread
From: Bernd Kuhls @ 2023-05-07 15:06 UTC (permalink / raw)
  To: buildroot

Am Sun, 7 May 2023 12:52:18 +0200 schrieb Yann E. MORIN:

>> Removed the option to disable uTP support, for details see upstream 
issue:
>> https://github.com/transmission/transmission/issues/4751
> 
> The option is still present, so why would we want to remove it?
> 
>     https://github.com/transmission/transmission/blob/main/
CMakeLists.txt
>     57  option(ENABLE_UTP "Build µTP support" ON)

Hi Yann,

because building without utp support is broken:

-- Configuring done
CMake Error at libtransmission/CMakeLists.txt:273 (target_link_libraries):
  Target "transmission" links to:

    libutp::libutp

  but the target was not found.  Possible reasons include:

libutp is required by https://github.com/transmission/transmission/blob/
0d3b321baca2cd02cc4b201826aed2b485e37c3f/libtransmission/
CMakeLists.txt#L285

This was discussed upstream in the forementioned issue 4751
https://github.com/transmission/transmission/issues/
4751#issuecomment-1464284585
"... but as a practical matter, no distros are going to disable UTP 
support. So I don't see an audience for this build option."

Getting rid of the libutp dependency in libtransmission was rejected btw:
https://github.com/transmission/transmission/pull/4882/

Regards, Bernd

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3
  2023-05-07 15:06     ` Bernd Kuhls
@ 2023-05-07 15:51       ` Yann E. MORIN
  0 siblings, 0 replies; 7+ messages in thread
From: Yann E. MORIN @ 2023-05-07 15:51 UTC (permalink / raw)
  To: Bernd Kuhls; +Cc: buildroot

Bernd, All,

On 2023-05-07 17:06 +0200, Bernd Kuhls spake thusly:
> Am Sun, 7 May 2023 12:52:18 +0200 schrieb Yann E. MORIN:
> >> Removed the option to disable uTP support, for details see upstream 
> issue:
> >> https://github.com/transmission/transmission/issues/4751
> > The option is still present, so why would we want to remove it?
> >     https://github.com/transmission/transmission/blob/main/CMakeLists.txt
> >     57  option(ENABLE_UTP "Build µTP support" ON)
> because building without utp support is broken:
> -- Configuring done
> CMake Error at libtransmission/CMakeLists.txt:273 (target_link_libraries):
>   Target "transmission" links to:
> 
>     libutp::libutp
> 
>   but the target was not found.  Possible reasons include:
> 
> libutp is required by https://github.com/transmission/transmission/blob/
> 0d3b321baca2cd02cc4b201826aed2b485e37c3f/libtransmission/
> CMakeLists.txt#L285
> 
> This was discussed upstream in the forementioned issue 4751
> https://github.com/transmission/transmission/issues/
> 4751#issuecomment-1464284585
> "... but as a practical matter, no distros are going to disable UTP 
> support. So I don't see an audience for this build option."

I did read the thread you mentioned, b efore I replied. But nowhere in
that thread is it mentionned that the option to build without was
actually removed, quite the opposite in fact.

First, @reardonia argued for keeping the option to build without libutp:

    https://github.com/transmission/transmission/issues/4751#issuecomment-1426548572

    Strongly prefer the WITH_UTP / ENABLE_UTP option. For a variety of
    safety issues, particularly with respect to private trackers, it's
    good to keep this disabled and reduce attack surface-area.

And eventually, @ckerr (a maintainer, I'd wager), seemed to change their
mind and accept @reardonia's request:

    https://github.com/transmission/transmission/issues/4751#issuecomment-1464284585

    @reardonia has done a lot of good patches and that sweat equity
    makes me look on this idea more favorably.

And finally, or completeness, here's the rest of the full comment you
partially quoted:

    but as a practical matter, no distros are going to disable UTP
    support. So I don't see an audience for this build option. And if
    that's the case, IDK if it makes sense to have it in transmission
    /transmission.

I.e. there is no explicit endorsement in that thread, that the option
has to be removed or not, and whether it was removed or not.

And in that CMakeLists.txt, one can also see:

    tr_allow_compile_if(
        ...
        [=[[ENABLE_UTP]]=]
            tr-utp.cc
        ...

    target_compile_definitions(${TR_NAME}
        PRIVATE
            ...
            $<$<BOOL:${ENABLE_UTP}>:WITH_UTP>
            ...

And all that seem to indicate that UTP support is still optional...

And indeed, the cmake option is still there (as I pointed out), but the
build is broken (as you pointed out).

So, maybe the build issue is really a bug?

In any case, just state something like that in the commit log:

    Even if the code seem to indicate that building without libutp or
    disabling UTP is possible, this cause build failures, so jsut make
    UTP mandatory.

Otherwise, all this is confusing.

Regards,
Yann E. MORIN.

> Getting rid of the libutp dependency in libtransmission was rejected btw:
> https://github.com/transmission/transmission/pull/4882/
> 
> Regards, Bernd
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH v2 1/2] package/libutp: bump version
  2023-05-06 16:05 [Buildroot] [PATCH v2 1/2] package/libutp: bump version Bernd Kuhls
  2023-05-06 16:05 ` [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3 Bernd Kuhls
@ 2023-05-07 16:18 ` Yann E. MORIN
  1 sibling, 0 replies; 7+ messages in thread
From: Yann E. MORIN @ 2023-05-07 16:18 UTC (permalink / raw)
  To: Bernd Kuhls; +Cc: Samuel Martin, buildroot

Bernd, All,

On 2023-05-06 18:05 +0200, Bernd Kuhls spake thusly:
> Needed for bumping transmission to version 4.0.1.
> 
> Added configure option to fix build error with gcc-12.
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
> v2: No changes
> 
>  package/libutp/libutp.hash | 2 +-
>  package/libutp/libutp.mk   | 3 ++-
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/package/libutp/libutp.hash b/package/libutp/libutp.hash
> index f347dc3f2c..0312f42612 100644
> --- a/package/libutp/libutp.hash
> +++ b/package/libutp/libutp.hash
> @@ -1,3 +1,3 @@
>  # Locally computed
> -sha256  484f802d044f774fa5f669476ecdd9cb6110c383781c39adc03bbf6575090771  libutp-059c9449a104e440e4f913756a5f560dd4ae76a9.tar.gz
> +sha256  d856fde68828d52eb39df40e15ad5dc4efaa9a51d4121bcbfbe47fed2163d20a  libutp-c95738b1a6644b919e5b64d3ea9736cfc5894e0b.tar.gz
>  sha256  73b400bf61dde52223dae1642205ca187f1a6b24a508dcce52bf386c27a0e58c  LICENSE
> diff --git a/package/libutp/libutp.mk b/package/libutp/libutp.mk
> index 394c1e9cb9..df0617555d 100644
> --- a/package/libutp/libutp.mk
> +++ b/package/libutp/libutp.mk
> @@ -4,10 +4,11 @@
>  #
>  ################################################################################
>  
> -LIBUTP_VERSION = 059c9449a104e440e4f913756a5f560dd4ae76a9
> +LIBUTP_VERSION = c95738b1a6644b919e5b64d3ea9736cfc5894e0b
>  LIBUTP_SITE = $(call github,transmission,libutp,$(LIBUTP_VERSION))
>  LIBUTP_LICENSE = MIT
>  LIBUTP_LICENSE_FILES = LICENSE
>  LIBUTP_INSTALL_STAGING = YES
> +LIBUTP_CONF_OPTS = -DLIBUTP_ENABLE_WERROR=OFF
>  
>  $(eval $(cmake-package))
> -- 
> 2.39.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3
  2023-05-06 16:05 ` [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3 Bernd Kuhls
  2023-05-07 10:52   ` Yann E. MORIN
       [not found]   ` <20230507105218.GH252090__6425.69195931486$1683456775$gmane$org@scaer>
@ 2023-05-07 16:23   ` Yann E. MORIN
  2 siblings, 0 replies; 7+ messages in thread
From: Yann E. MORIN @ 2023-05-07 16:23 UTC (permalink / raw)
  To: Bernd Kuhls; +Cc: Samuel Martin, buildroot

Bernd, All,

On 2023-05-06 18:05 +0200, Bernd Kuhls spake thusly:
> This bump depends on bumping libutp to at least
> c95738b1a6644b919e5b64d3ea9736cfc5894e0b.
> 
> Updated license hash due to copyright year bump:
> https://github.com/transmission/transmission/commit/4b8cfa2e576feea6a6f2fd5ae5fe1a59449aca6f
> 
> Switched buildsystem to cmake, reworked dependencies.
> 
> Removed all patches, transmission now depends on the libutp package and
> uses cmake as build system.
> 
> Upstream removed the configure option for lightweight build:
> https://github.com/transmission/transmission/commit/47ebb3f63a2f8983f9858bfbec6b72c42588a388
> 
> Removed the option to disable uTP support, for details see upstream issue:
> https://github.com/transmission/transmission/issues/4751

I reworded that sentence with the results of our discussion in the
thread.

[--SNIP--]
> diff --git a/package/transmission/0001-fix-utypes.patch b/package/transmission/0001-fix-utypes.patch
> deleted file mode 100644
> index 81efa884a7..0000000000
> --- a/package/transmission/0001-fix-utypes.patch
> +++ /dev/null
> @@ -1,30 +0,0 @@
> -Fix build error "conflicting types for 'UTP_Write'"
> -
> -Fetch from: http://www.adminsehow.com/2012/10/how-to-install-transmission-2-71-on-debian-6-0-squeeze/
> -
> -The corresponding upstream bug report (unfixed) can be found here:
> -https://trac.transmissionbt.com/ticket/5232
> -
> -Signed-off-by: Bernd Kuhls <berndkuhls@hotmail.com>
> -[Fabrice: adapt to 3.00, which always uses stdbool.h]
> -Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -[yann.morin.1998@free.fr:
> -  - don't comment-out dropped code, just drop it
> -  - use CR-LF in replacement code, as used by upstream
> -]
> -Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> -
> ---- transmission-2.82.org/third-party/libutp/utypes.h	2013-08-09 04:47:43.000000000 +0200
> -+++ transmission-2.82/third-party/libutp/utypes.h	2014-01-02 20:17:18.000000000 +0100
> -@@ -35,8 +35,8 @@
> - typedef const char * cstr;
> - typedef char * str;
> - 
> --#ifndef __cplusplus
> --typedef uint8 bool;
> --#endif
> -+#ifndef __cplusplus
> -+#include <stdbool.h>
> -+#endif
> - 
> - #endif //__UTYPES_H__

This hunk fails to apply:

    error: patch failed: package/transmission/0001-fix-utypes.patch:1
    error: package/transmission/0001-fix-utypes.patch: patch does not apply

I guess that's because there are CRLF, and patchwork borked them
(again).

So I manually fixed it before comitting, please check that I did not
bork it too much.

Applied to master, thanks.

Regards,
Yann E. MORIN.

> diff --git a/package/transmission/0002-fix-utp-include.patch b/package/transmission/0002-fix-utp-include.patch
> deleted file mode 100644
> index db8b5ab39b..0000000000
> --- a/package/transmission/0002-fix-utp-include.patch
> +++ /dev/null
> @@ -1,46 +0,0 @@
> -Fix compilation when using system libminiupnpc with libutp disabled
> -
> -If transmission uses its own copy of libminiupnpc in third-party/ then
> -the configure script adds that path to CFLAGS:
> -https://trac.transmissionbt.com/browser/trunk/configure.ac#L455
> -
> -This path is also added if transmission uses its own copy of libutp:
> -https://trac.transmissionbt.com/browser/trunk/configure.ac#L349
> -
> -But if a system-provided libminiupnpc is used and utp-support is
> -disabled using --disable-utp the result is a build error, detected by
> -buildroot autobuilders:
> -http://autobuild.buildroot.net/results/b79/b79e51140cb0320554a58ce59dcb33f53cd9211a/build-end.log
> -
> -net.c:40:24: fatal error: libutp/utp.h: No such file or directory
> -
> -Patch proposed by Yann E. MORIN:
> -http://lists.busybox.net/pipermail/buildroot/2016-July/167896.html
> -
> -Patch sent upstream: https://trac.transmissionbt.com/ticket/6163
> -
> -Signed-off-by: Bernd Kuhls <berndkuhls@hotmail.com>
> -
> -diff -uNr transmission-2.92.org/configure.ac transmission-2.92/configure.ac
> ---- transmission-2.92.org/configure.ac	2016-03-06 21:23:45.336198393 +0100
> -+++ transmission-2.92/configure.ac	2016-07-31 12:13:40.921660078 +0200
> -@@ -358,6 +358,10 @@
> -     else
> -       AC_MSG_ERROR("Unable to build uTP support -- C++ compiler not found")
> -     fi
> -+else
> -+    # For the system libminiupnpc with UTC disabled,
> -+    # point to our bundled UTP headers
> -+    LIBUTP_CFLAGS="-I\$(top_srcdir)/third-party/"
> - fi
> - AC_SUBST(LIBUTP_CFLAGS)
> - AC_SUBST(LIBUTP_LIBS)
> -@@ -457,7 +461,7 @@
> -     LIBUPNP_LIBS_QT="\$\${TRANSMISSION_TOP}/third-party/miniupnp/libminiupnp.a"
> - else
> -     AC_DEFINE([SYSTEM_MINIUPNP])
> --    LIBUPNP_CFLAGS="" 
> -+    LIBUPNP_CFLAGS="${LIBUTP_CFLAGS}"
> -     LIBUPNP_LIBS="-lminiupnpc"
> -     LIBUPNP_LIBS_QT="-lminiupnpc"
> - fi
> diff --git a/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch b/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch
> deleted file mode 100644
> index bfb0084bc4..0000000000
> --- a/package/transmission/0003-configure.ac-fix-autoreconf-with-autoconf-2.70.patch
> +++ /dev/null
> @@ -1,42 +0,0 @@
> -From 857ac91a03a096601f62b358fc61355d4cc25b9b Mon Sep 17 00:00:00 2001
> -From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -Date: Sat, 9 Oct 2021 18:51:13 +0200
> -Subject: [PATCH] configure.ac: fix autoreconf with autoconf >= 2.70
> -
> -Remove the whitespace before IT_PROG_INTLTOOL as suggested in
> -https://www.mail-archive.com/bug-autoconf@gnu.org/msg04851.html
> -to avoid the following build failure with autoconf >= 2.70:
> -
> -ERROR: 'IT_PROG_INTLTOOL' must appear in configure.ac for intltool to work.
> -
> -The problem has been introduced by upstream autoconf commit
> -http://git.savannah.gnu.org/gitweb/?p=autoconf.git;a=commit;h=b3b3af821a0b9e82ebe56b77b1731d2fe58f52de
> -
> -Fixes:
> - - http://autobuild.buildroot.org/results/e8bea26f6b12adf16335b24836a7c5c31911af6a
> -
> -Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -[Upstream status: not upstreamable as upstream swicthed to cmake (
> -https://github.com/transmission/transmission/issues/1573)]
> ----
> - configure.ac | 4 +---
> - 1 file changed, 1 insertion(+), 3 deletions(-)
> -
> -diff --git a/configure.ac b/configure.ac
> -index fd1132fda..04e0e19bb 100644
> ---- a/configure.ac
> -+++ b/configure.ac
> -@@ -555,9 +555,7 @@ dnl it should be safe to re-edit 0.40 back down to 0.23
> - use_nls=no
> - if test "x$enable_nls" = "xyes" ; then
> -     use_nls=yes
> --    m4_ifdef([IT_PROG_INTLTOOL],
> --             [IT_PROG_INTLTOOL([0.35.0],[no-xml])],
> --             [AC_MSG_ERROR("--enable-nls requires intltool to be installed.")])
> -+IT_PROG_INTLTOOL([0.35.0],[no-xml])
> -     AC_CHECK_HEADERS([libintl.h])
> -     GETTEXT_PACKAGE=transmission-gtk
> -     AC_SUBST(GETTEXT_PACKAGE)
> --- 
> -2.33.0
> -
> diff --git a/package/transmission/Config.in b/package/transmission/Config.in
> index 980b042e33..84bd85eeff 100644
> --- a/package/transmission/Config.in
> +++ b/package/transmission/Config.in
> @@ -1,12 +1,23 @@
>  config BR2_PACKAGE_TRANSMISSION
>  	bool "transmission"
> +	depends on !BR2_BINFMT_FLAT # libnatpmp
> +	depends on BR2_INSTALL_LIBSTDCPP # libutp
> +	depends on !BR2_STATIC_LIBS # libnatpmp
> +	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_7 # C++17
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
>  	depends on BR2_USE_MMU # fork()
> -	select BR2_PACKAGE_ZLIB
> -	select BR2_PACKAGE_OPENSSL
> -	select BR2_PACKAGE_LIBOPENSSL_ENABLE_RC4 if BR2_PACKAGE_LIBOPENSSL
> +	depends on BR2_USE_WCHAR # libpsl
> +	select BR2_PACKAGE_DHT
> +	select BR2_PACKAGE_LIBB64
>  	select BR2_PACKAGE_LIBCURL
> +	select BR2_PACKAGE_LIBDEFLATE
>  	select BR2_PACKAGE_LIBEVENT
> +	select BR2_PACKAGE_LIBMINIUPNPC
> +	select BR2_PACKAGE_LIBNATPMP
> +	select BR2_PACKAGE_LIBPSL
> +	select BR2_PACKAGE_LIBUTP
> +	select BR2_PACKAGE_OPENSSL
> +	select BR2_PACKAGE_ZLIB
>  	help
>  	  Transmission is a cross-platform BitTorrent client.
>  
> @@ -14,15 +25,6 @@ config BR2_PACKAGE_TRANSMISSION
>  
>  if BR2_PACKAGE_TRANSMISSION
>  
> -config BR2_PACKAGE_TRANSMISSION_UTP
> -	bool "Transmission uTP support"
> -	depends on BR2_INSTALL_LIBSTDCPP
> -	help
> -	  Install support for uTorrent Transport Protocol
> -
> -comment "uTP needs a toolchain w/ C++"
> -	depends on !BR2_INSTALL_LIBSTDCPP
> -
>  config BR2_PACKAGE_TRANSMISSION_CLI
>  	bool "transmission-cli"
>  	help
> @@ -43,6 +45,8 @@ comment "transmission-gtk needs NLS enabled"
>  	depends on BR2_PACKAGE_LIBGTK3 && !BR2_SYSTEM_ENABLE_NLS
>  endif
>  
> -comment "transmission needs a toolchain w/ threads"
> +comment "transmission needs a toolchain w/ dynamic library, threads, C++, gcc >= 7"
>  	depends on BR2_USE_MMU
> -	depends on !BR2_TOOLCHAIN_HAS_THREADS
> +	depends on !BR2_BINFMT_FLAT
> +	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_INSTALL_LIBSTDCPP || \
> +		!BR2_TOOLCHAIN_GCC_AT_LEAST_7 || !BR2_STATIC_LIBS
> diff --git a/package/transmission/transmission.hash b/package/transmission/transmission.hash
> index 13cbc969d3..e6775c47e7 100644
> --- a/package/transmission/transmission.hash
> +++ b/package/transmission/transmission.hash
> @@ -1,3 +1,4 @@
> +# From https://transmissionbt.com/download
> +sha256  b6b01fd58e42bb14f7aba0253db932ced050fcd2bba5d9f8469d77ddd8ad545a  transmission-4.0.3.tar.xz
>  # Locally calculated
> -sha256  9144652fe742f7f7dd6657716e378da60b751aaeda8bef8344b3eefc4db255f2  transmission-3.00.tar.xz
> -sha256  2b16da5bd02790d1e4d332cfd42b9dfd7d1b0ed3a16a75dc7e76b6ed239fdba9  COPYING
> +sha256  f56732960a61ecf3b9637404eef1a39221d2006336a98792b7b65a79f155449c  COPYING
> diff --git a/package/transmission/transmission.mk b/package/transmission/transmission.mk
> index 1660d2c2b9..f3a200ced3 100644
> --- a/package/transmission/transmission.mk
> +++ b/package/transmission/transmission.mk
> @@ -4,55 +4,54 @@
>  #
>  ################################################################################
>  
> -TRANSMISSION_VERSION = 3.00
> -TRANSMISSION_SITE = https://github.com/transmission/transmission-releases/raw/master
> +TRANSMISSION_VERSION = 4.0.3
>  TRANSMISSION_SOURCE = transmission-$(TRANSMISSION_VERSION).tar.xz
> +TRANSMISSION_SITE = https://github.com/transmission/transmission/releases/download/$(TRANSMISSION_VERSION)
>  TRANSMISSION_DEPENDENCIES = \
>  	host-pkgconf \
> -	host-intltool \
> +	dht \
> +	libb64 \
>  	libcurl \
> +	libdeflate \
>  	libevent \
> +	libminiupnpc \
> +	libnatpmp \
> +	libpsl \
> +	libutp \
>  	openssl \
>  	zlib
> -TRANSMISSION_AUTORECONF = YES
>  TRANSMISSION_CONF_OPTS = \
> -	--without-inotify \
> -	--enable-lightweight
> +	-DENABLE_TESTS=OFF \
> +	-DRUN_CLANG_TIDY=OFF \
> +	-DUSE_SYSTEM_B64=ON \
> +	-DUSE_SYSTEM_DEFLATE=ON \
> +	-DUSE_SYSTEM_DHT=ON \
> +	-DUSE_SYSTEM_NATPMP=ON \
> +	-DUSE_SYSTEM_PSL=ON \
> +	-DWITH_INOTIFY=OFF
>  TRANSMISSION_LICENSE = GPL-2.0 or GPL-3.0 with OpenSSL exception
>  TRANSMISSION_LICENSE_FILES = COPYING
>  TRANSMISSION_CPE_ID_VENDOR = transmissionbt
>  
> -ifeq ($(BR2_PACKAGE_LIBMINIUPNPC),y)
> -TRANSMISSION_DEPENDENCIES += libminiupnpc
> -endif
> -
> -ifeq ($(BR2_PACKAGE_LIBNATPMP),y)
> -TRANSMISSION_DEPENDENCIES += libnatpmp
> -TRANSMISSION_CONF_OPTS += --enable-external-natpmp
> -else
> -TRANSMISSION_CONF_OPTS += --disable-external-natpmp
> -endif
> -
> -ifeq ($(BR2_PACKAGE_TRANSMISSION_UTP),y)
> -TRANSMISSION_CONF_OPTS += --enable-utp
> -else
> -TRANSMISSION_CONF_OPTS += --disable-utp
> +# Uses __atomic_load_8
> +ifeq ($(BR2_TOOLCHAIN_HAS_LIBATOMIC),y)
> +TRANSMISSION_CONF_OPTS += -DCMAKE_EXE_LINKER_FLAGS=-latomic
>  endif
>  
>  ifeq ($(BR2_PACKAGE_TRANSMISSION_CLI),y)
> -TRANSMISSION_CONF_OPTS += --enable-cli
> +TRANSMISSION_CONF_OPTS += -DENABLE_CLI=ON
>  else
> -TRANSMISSION_CONF_OPTS += --disable-cli
> +TRANSMISSION_CONF_OPTS += -DENABLE_CLI=OFF
>  endif
>  
>  ifeq ($(BR2_PACKAGE_TRANSMISSION_DAEMON),y)
> -TRANSMISSION_CONF_OPTS += --enable-daemon
> +TRANSMISSION_CONF_OPTS += -DENABLE_DAEMON=ON
>  
>  ifeq ($(BR2_PACKAGE_SYSTEMD),y)
>  TRANSMISSION_DEPENDENCIES += systemd
> -TRANSMISSION_CONF_OPTS += --with-systemd
> +TRANSMISSION_CONF_OPTS += -DWITH_SYSTEMD=ON
>  else
> -TRANSMISSION_CONF_OPTS += --without-systemd
> +TRANSMISSION_CONF_OPTS += -DWITH_SYSTEMD=OFF
>  endif
>  
>  define TRANSMISSION_USERS
> @@ -70,14 +69,14 @@ define TRANSMISSION_INSTALL_INIT_SYSTEMD
>  endef
>  
>  else
> -TRANSMISSION_CONF_OPTS += --disable-daemon
> +TRANSMISSION_CONF_OPTS += -DENABLE_DAEMON=OFF
>  endif
>  
>  ifeq ($(BR2_PACKAGE_TRANSMISSION_GTK),y)
> -TRANSMISSION_CONF_OPTS += --with-gtk
> +TRANSMISSION_CONF_OPTS += -DENABLE_GTK=ON
>  TRANSMISSION_DEPENDENCIES += libgtk3
>  else
> -TRANSMISSION_CONF_OPTS += --without-gtk
> +TRANSMISSION_CONF_OPTS += -DENABLE_GTK=OFF
>  endif
>  
> -$(eval $(autotools-package))
> +$(eval $(cmake-package))
> -- 
> 2.39.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-05-07 16:24 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-06 16:05 [Buildroot] [PATCH v2 1/2] package/libutp: bump version Bernd Kuhls
2023-05-06 16:05 ` [Buildroot] [PATCH v2 2/2] package/transmission: bump version to 4.0.3 Bernd Kuhls
2023-05-07 10:52   ` Yann E. MORIN
     [not found]   ` <20230507105218.GH252090__6425.69195931486$1683456775$gmane$org@scaer>
2023-05-07 15:06     ` Bernd Kuhls
2023-05-07 15:51       ` Yann E. MORIN
2023-05-07 16:23   ` Yann E. MORIN
2023-05-07 16:18 ` [Buildroot] [PATCH v2 1/2] package/libutp: bump version Yann E. MORIN

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox