[Buildroot] [PATCH/next 1/1] package/libbluray: add optional support for libudfread

[Buildroot] [PATCH/next 1/1] package/libbluray: add optional support for libudfread

[Bernd Kuhls]

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/libbluray/0001-libudfread.patch | 29 +++++++++++++++++++++++++
 package/libbluray/libbluray.mk          |  6 +++++
 2 files changed, 35 insertions(+)
 create mode 100644 package/libbluray/0001-libudfread.patch

diff --git a/package/libbluray/0001-libudfread.patch b/package/libbluray/0001-libudfread.patch
new file mode 100644
index 0000000000..5cd9ab8111
--- /dev/null
+++ b/package/libbluray/0001-libudfread.patch
@@ -0,0 +1,29 @@
+From 24d38b8ec371d89b51f8b65f3085672db12334ff Mon Sep 17 00:00:00 2001
+From: Sebastian Ramacher <sramacher@debian.org>
+Date: Sun, 20 Dec 2020 13:32:33 +0100
+Subject: [PATCH] Update check for new libudfread pkg-config file name
+
+Downloaded from upstream commit:
+https://code.videolan.org/videolan/libbluray/-/commit/24d38b8ec371d89b51f8b65f3085672db12334ff
+
+Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
+---
+ configure.ac | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index e0e80cce..250db73b 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -279,7 +279,7 @@ AM_CONDITIONAL([BDJ_J2SE], [ test x"$BDJ_TYPE" != x"j2me" ])
+ dnl bootclasspath
+ AC_SUBST(BDJ_BOOTCLASSPATH)
+ 
+-PKG_CHECK_MODULES([LIBUDFREAD], [udfread >= 1.1.0],
++PKG_CHECK_MODULES([LIBUDFREAD], [libudfread >= 1.1.0],
+     [with_libudfread=yes
+      AC_DEFINE([HAVE_LIBUDFREAD], [1], [Define to 1 if external libudfread is to be used])
+      PACKAGES="$PACKAGES libudfread >= 1.1.0"
+-- 
+GitLab
+
diff --git a/package/libbluray/libbluray.mk b/package/libbluray/libbluray.mk
index 36f59190b1..e4fca5204f 100644
--- a/package/libbluray/libbluray.mk
+++ b/package/libbluray/libbluray.mk
@@ -12,6 +12,8 @@ LIBBLURAY_LICENSE = LGPL-2.1+
 LIBBLURAY_LICENSE_FILES = COPYING
 LIBBLURAY_CPE_ID_VENDOR = videolan
 LIBBLURAY_DEPENDENCIES = host-pkgconf
+# 0001-libudfread.patch
+LIBBLURAY_AUTORECONF = YES
 
 LIBBLURAY_CONF_OPTS = --disable-bdjava-jar
 
@@ -19,6 +21,10 @@ ifeq ($(BR2_PACKAGE_LIBICONV),y)
 LIBBLURAY_DEPENDENCIES += libiconv
 endif
 
+ifeq ($(BR2_PACKAGE_LIBUDFREAD),y)
+LIBBLURAY_DEPENDENCIES += libudfread
+endif
+
 ifeq ($(BR2_PACKAGE_FREETYPE),y)
 LIBBLURAY_DEPENDENCIES += freetype
 LIBBLURAY_CONF_OPTS += --with-freetype
-- 
2.29.2

[Buildroot] [PATCH/next 1/1] package/libbluray: add optional support for libudfread

[Yann E. MORIN]

Bernd, All,

On 2021-02-21 10:35 +0100, Bernd Kuhls spake thusly:
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

This patch looks like a feature patch. It is indeed a backport from
upstream, which is good, but your commit log should explain why we need
to do that backport.

Regards,
Yann E. MORIN.

> ---
>  package/libbluray/0001-libudfread.patch | 29 +++++++++++++++++++++++++
>  package/libbluray/libbluray.mk          |  6 +++++
>  2 files changed, 35 insertions(+)
>  create mode 100644 package/libbluray/0001-libudfread.patch
> 
> diff --git a/package/libbluray/0001-libudfread.patch b/package/libbluray/0001-libudfread.patch
> new file mode 100644
> index 0000000000..5cd9ab8111
> --- /dev/null
> +++ b/package/libbluray/0001-libudfread.patch
> @@ -0,0 +1,29 @@
> +From 24d38b8ec371d89b51f8b65f3085672db12334ff Mon Sep 17 00:00:00 2001
> +From: Sebastian Ramacher <sramacher@debian.org>
> +Date: Sun, 20 Dec 2020 13:32:33 +0100
> +Subject: [PATCH] Update check for new libudfread pkg-config file name
> +
> +Downloaded from upstream commit:
> +https://code.videolan.org/videolan/libbluray/-/commit/24d38b8ec371d89b51f8b65f3085672db12334ff
> +
> +Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> +---
> + configure.ac | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/configure.ac b/configure.ac
> +index e0e80cce..250db73b 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -279,7 +279,7 @@ AM_CONDITIONAL([BDJ_J2SE], [ test x"$BDJ_TYPE" != x"j2me" ])
> + dnl bootclasspath
> + AC_SUBST(BDJ_BOOTCLASSPATH)
> + 
> +-PKG_CHECK_MODULES([LIBUDFREAD], [udfread >= 1.1.0],
> ++PKG_CHECK_MODULES([LIBUDFREAD], [libudfread >= 1.1.0],
> +     [with_libudfread=yes
> +      AC_DEFINE([HAVE_LIBUDFREAD], [1], [Define to 1 if external libudfread is to be used])
> +      PACKAGES="$PACKAGES libudfread >= 1.1.0"
> +-- 
> +GitLab
> +
> diff --git a/package/libbluray/libbluray.mk b/package/libbluray/libbluray.mk
> index 36f59190b1..e4fca5204f 100644
> --- a/package/libbluray/libbluray.mk
> +++ b/package/libbluray/libbluray.mk
> @@ -12,6 +12,8 @@ LIBBLURAY_LICENSE = LGPL-2.1+
>  LIBBLURAY_LICENSE_FILES = COPYING
>  LIBBLURAY_CPE_ID_VENDOR = videolan
>  LIBBLURAY_DEPENDENCIES = host-pkgconf
> +# 0001-libudfread.patch
> +LIBBLURAY_AUTORECONF = YES
>  
>  LIBBLURAY_CONF_OPTS = --disable-bdjava-jar
>  
> @@ -19,6 +21,10 @@ ifeq ($(BR2_PACKAGE_LIBICONV),y)
>  LIBBLURAY_DEPENDENCIES += libiconv
>  endif
>  
> +ifeq ($(BR2_PACKAGE_LIBUDFREAD),y)
> +LIBBLURAY_DEPENDENCIES += libudfread
> +endif
> +
>  ifeq ($(BR2_PACKAGE_FREETYPE),y)
>  LIBBLURAY_DEPENDENCIES += freetype
>  LIBBLURAY_CONF_OPTS += --with-freetype
> -- 
> 2.29.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/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.  |
'------------------------------^-------^------------------^--------------------'