Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/libopenaptx: drop package
@ 2021-12-19 11:04 Yann E. MORIN
  2021-12-19 11:39 ` Yunhao Tian
  2021-12-20 20:36 ` Arnout Vandecappelle
  0 siblings, 2 replies; 7+ messages in thread
From: Yann E. MORIN @ 2021-12-19 11:04 UTC (permalink / raw)
  To: buildroot; +Cc: Yann E. MORIN, Thomas Petazzoni, Yunhao Tian, Fabrice Fontaine

The licensing about libopenaptx is very complicated. Version 0.2.1 was
tagged with the sole purpose of relicensing from LGPLv2.1+ to GPLv3.0+:

    https://github.com/pali/libopenaptx/commit/811bc18586d634042618d633727ac0281d4170b8

However, this is has various issues:

  - the code originally comes from ffmpeg, where it was LGPLv2.1+, so
    the relicensing is dubious at best;

  - an explicit ban to f.d.o and Collabora from using that code, and
    with a long rant about it, even stating that the license to f.d.o or
    Collabora was terminated:
        https://github.com/pali/libopenaptx/commit/811bc18586d634042618d633727ac0281d4170b8#commitcomment-54154156

        Per §8/LGPL and §8/GPL license was terminated with Collabora and
        Freedesktop and was not automatically reinstated. It applies to
        all versions of this library, including older and Collabora and
        Freedesktop cannot use this library anymore.

A set of upstream issues have been raised, in relation with this topic:

    https://github.com/pali/libopenaptx/issues/11
        Possible license violation

    https://github.com/pali/libopenaptx/issues/12
        Restrictions being invalidated by terms of GPL

    https://github.com/pali/libopenaptx/issues/13
        Please provide links or citations for README claims

In all cases, the author dismissed those as invalid, with various levels
of amiability...

This situation is very concerning.

Drop the package to avoid any issue.

Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
Cc: Yunhao Tian <t123yh.xyz@gmail.com>
Cc: Peter Korsgaard <peter@korsgaard.com>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Cc: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 DEVELOPERS                           |  3 ---
 package/Config.in                    |  1 -
 package/bluez-alsa/bluez-alsa.mk     |  7 -------
 package/libopenaptx/Config.in        | 14 --------------
 package/libopenaptx/libopenaptx.hash |  3 ---
 package/libopenaptx/libopenaptx.mk   | 29 ----------------------------
 6 files changed, 57 deletions(-)
 delete mode 100644 package/libopenaptx/Config.in
 delete mode 100644 package/libopenaptx/libopenaptx.hash
 delete mode 100644 package/libopenaptx/libopenaptx.mk

diff --git a/DEVELOPERS b/DEVELOPERS
index 35f3be81b8..ecf12f2e16 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -2966,9 +2966,6 @@ F:	support/testing/tests/package/test_libftdi1.py
 F:	support/testing/tests/package/test_python_can.py
 F:	utils/scanpypi
 
-N:	Yunhao Tian <t123yh.xyz@gmail.com>
-F:	package/libopenaptx/
-
 N:	Zoltan Gyarmati <zgyarmati@zgyarmati.de>
 F:	package/crudini/
 F:	package/grantlee/
diff --git a/package/Config.in b/package/Config.in
index bcb23132c9..c61ef472e3 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1744,7 +1744,6 @@ menu "Multimedia"
 	source "package/libmms/Config.in"
 	source "package/libmpeg2/Config.in"
 	source "package/libogg/Config.in"
-	source "package/libopenaptx/Config.in"
 	source "package/libopenh264/Config.in"
 	source "package/libopusenc/Config.in"
 	source "package/libtheora/Config.in"
diff --git a/package/bluez-alsa/bluez-alsa.mk b/package/bluez-alsa/bluez-alsa.mk
index f5e3aa1245..f81720f767 100644
--- a/package/bluez-alsa/bluez-alsa.mk
+++ b/package/bluez-alsa/bluez-alsa.mk
@@ -69,11 +69,4 @@ else
 BLUEZ_ALSA_CONF_OPTS += --disable-rfcomm
 endif
 
-ifeq ($(BR2_PACKAGE_LIBOPENAPTX),y)
-BLUEZ_ALSA_DEPENDENCIES += libopenaptx
-BLUEZ_ALSA_CONF_OPTS += --with-libopenaptx --enable-aptx --enable-aptx-hd
-else
-BLUEZ_ALSA_CONF_OPTS += --without-libopenaptx --disable-aptx --disable-aptx-hd
-endif
-
 $(eval $(autotools-package))
diff --git a/package/libopenaptx/Config.in b/package/libopenaptx/Config.in
deleted file mode 100644
index 46ff08a2d1..0000000000
--- a/package/libopenaptx/Config.in
+++ /dev/null
@@ -1,14 +0,0 @@
-config BR2_PACKAGE_LIBOPENAPTX
-	bool "libopenaptx"
-	# unconditionallly builds a shared lib
-	depends on !BR2_STATIC_LIBS
-	help
-	  This is Open Source implementation of Audio Processing
-	  Technology codec (aptX) originally derived from ffmpeg 4.0
-	  project and licensed under GPLv3+. This codec is mainly used
-	  in Bluetooth A2DP profile.
-
-	  https://github.com/pali/libopenaptx
-
-comment "libopenaptx needs a toolchain w/ dynamic library"
-	depends on BR2_STATIC_LIBS
diff --git a/package/libopenaptx/libopenaptx.hash b/package/libopenaptx/libopenaptx.hash
deleted file mode 100644
index 76b4e66c9b..0000000000
--- a/package/libopenaptx/libopenaptx.hash
+++ /dev/null
@@ -1,3 +0,0 @@
-# Locally calculated:
-sha256 8ceb4b9ee5adedde47b31e975c1d90c73ad27b6b165a1dcd80c7c545eb65b903 COPYING
-sha256 a19b3b642def6f86441b73dd4efda6e6f13d49e7e2d6e5e91cc7eb464ebb505a libopenaptx-0.2.1.tar.gz
diff --git a/package/libopenaptx/libopenaptx.mk b/package/libopenaptx/libopenaptx.mk
deleted file mode 100644
index c2f5fa2390..0000000000
--- a/package/libopenaptx/libopenaptx.mk
+++ /dev/null
@@ -1,29 +0,0 @@
-################################################################################
-#
-# libopenaptx
-#
-################################################################################
-
-LIBOPENAPTX_VERSION = 0.2.1
-LIBOPENAPTX_SITE = $(call github,pali,libopenaptx,$(LIBOPENAPTX_VERSION))
-LIBOPENAPTX_LICENSE = GPL-3.0+
-LIBOPENAPTX_LICENSE_FILES = COPYING
-LIBOPENAPTX_INSTALL_STAGING = YES
-
-define LIBOPENAPTX_BUILD_CMDS
-	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS)
-endef
-
-define LIBOPENAPTX_INSTALL_TARGET_CMDS
-	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS) \
-		DESTDIR=$(TARGET_DIR) \
-		PREFIX=/usr install
-endef
-
-define LIBOPENAPTX_INSTALL_STAGING_CMDS
-	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS) \
-		DESTDIR=$(STAGING_DIR) \
-		PREFIX=/usr install
-endef
-
-$(eval $(generic-package))
-- 
2.25.1

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

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2021-12-21 21:46 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-12-19 11:04 [Buildroot] [PATCH] package/libopenaptx: drop package Yann E. MORIN
2021-12-19 11:39 ` Yunhao Tian
2021-12-20 20:36 ` Arnout Vandecappelle
2021-12-20 22:08   ` Yann E. MORIN
2021-12-20 22:30     ` Arnout Vandecappelle
2021-12-21 21:19       ` Yann E. MORIN
2021-12-21 21:46         ` Arnout Vandecappelle

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