diff mbox series

[1/1] package/libpsl: bump to version 0.21.2

Message ID 20230114175446.1352883-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series [1/1] package/libpsl: bump to version 0.21.2 | expand

Commit Message

Fabrice Fontaine Jan. 14, 2023, 5:54 p.m. UTC
- Drop patch (already in version)
- Update hash of COPYING (update in year:
  https://github.com/rockdaboot/libpsl/commit/927b202c69bcd9be4c30ba51c74925f2575e1a69)
- builtin is a boolean since
  https://github.com/rockdaboot/libpsl/commit/aa4909766c24c17bd1d9000ca419f6dc6b32c238

https://github.com/rockdaboot/libpsl/releases/tag/0.21.2

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 ...dd-libiconv-dep-to-meson-build-again.patch | 60 -------------------
 package/libpsl/libpsl.hash                    |  4 +-
 package/libpsl/libpsl.mk                      |  9 +--
 3 files changed, 7 insertions(+), 66 deletions(-)
 delete mode 100644 package/libpsl/0001-Add-libiconv-dep-to-meson-build-again.patch

Comments

Adrian Perez de Castro Jan. 14, 2023, 10:27 p.m. UTC | #1
Hi,

Changes LGTM.

On Sat, 14 Jan 2023 18:54:46 +0100 Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:
> - Drop patch (already in version)
> - Update hash of COPYING (update in year:
>   https://github.com/rockdaboot/libpsl/commit/927b202c69bcd9be4c30ba51c74925f2575e1a69)
> - builtin is a boolean since
>   https://github.com/rockdaboot/libpsl/commit/aa4909766c24c17bd1d9000ca419f6dc6b32c238
> 
> https://github.com/rockdaboot/libpsl/releases/tag/0.21.2
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Reviewed-by: Adrian Perez de Castro <aperez@igalia.com>

> ---
>  ...dd-libiconv-dep-to-meson-build-again.patch | 60 -------------------
>  package/libpsl/libpsl.hash                    |  4 +-
>  package/libpsl/libpsl.mk                      |  9 +--
>  3 files changed, 7 insertions(+), 66 deletions(-)
>  delete mode 100644 package/libpsl/0001-Add-libiconv-dep-to-meson-build-again.patch
> 
> diff --git a/package/libpsl/0001-Add-libiconv-dep-to-meson-build-again.patch b/package/libpsl/0001-Add-libiconv-dep-to-meson-build-again.patch
> deleted file mode 100644
> index 72f50aa593..0000000000
> --- a/package/libpsl/0001-Add-libiconv-dep-to-meson-build-again.patch
> +++ /dev/null
> @@ -1,60 +0,0 @@
> -From 5b63816e82287808e81b8603a528af3cc6fab354 Mon Sep 17 00:00:00 2001
> -From: Zhao Zhili <zhilizhao@tencent.com>
> -Date: Thu, 7 Jan 2021 20:59:17 +0800
> -Subject: [PATCH] Add libiconv dep to meson build again
> -
> -Build success on macOS and ubuntu.
> -
> -[Retrieved from:
> -https://github.com/rockdaboot/libpsl/commit/5b63816e82287808e81b8603a528af3cc6fab354]
> -Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ----
> - meson.build     | 14 ++++++++++++++
> - src/meson.build |  2 +-
> - 2 files changed, 15 insertions(+), 1 deletion(-)
> -
> -diff --git a/meson.build b/meson.build
> -index f248171..fb2de31 100644
> ---- a/meson.build
> -+++ b/meson.build
> -@@ -20,6 +20,7 @@ libicu_dep = notfound
> - libidn_dep = notfound
> - libunistring = notfound
> - networking_deps = notfound
> -+libiconv_dep = notfound
> - 
> - # FIXME: Cleanup this when Meson gets 'feature-combo':
> - # https://github.com/mesonbuild/meson/issues/4566
> -@@ -86,6 +87,19 @@ endif
> - if libidn2_dep.found() or libidn_dep.found()
> -   # Check for libunistring, we need it for psl_str_to_utf8lower()
> -   libunistring = cc.find_library('unistring')
> -+  found_iconv = false
> -+  if cc.has_function('iconv_open')
> -+    libiconv_dep = []
> -+    found_iconv = true
> -+  endif
> -+  if not found_iconv and cc.has_header_symbol('iconv.h', 'iconv_open')
> -+    libiconv_dep = [cc.find_library('iconv')]
> -+    found_iconv = true
> -+  endif
> -+
> -+  if not found_iconv
> -+    error('iconv implementation not found')
> -+  endif
> - endif
> - 
> - if host_machine.system() == 'windows'
> -diff --git a/src/meson.build b/src/meson.build
> -index 0d1dcdf..746bf44 100644
> ---- a/src/meson.build
> -+++ b/src/meson.build
> -@@ -19,7 +19,7 @@ cargs = [
> - libpsl = library('psl', sources, suffixes_dafsa_h,
> -   include_directories : [configinc, includedir],
> -   c_args : cargs,
> --  dependencies : [libidn2_dep, libidn_dep, libicu_dep, libunistring, networking_deps],
> -+  dependencies : [libidn2_dep, libidn_dep, libicu_dep, libunistring, networking_deps, libiconv_dep],
> -   version: lt_version,
> -   install: true,
> - )
> diff --git a/package/libpsl/libpsl.hash b/package/libpsl/libpsl.hash
> index 552935e9ec..a35a609ff2 100644
> --- a/package/libpsl/libpsl.hash
> +++ b/package/libpsl/libpsl.hash
> @@ -1,6 +1,6 @@
>  # Locally generated
> -sha256  ac6ce1e1fbd4d0254c4ddb9d37f1fa99dec83619c1253328155206b896210d4c  libpsl-0.21.1.tar.gz
> +sha256  e35991b6e17001afa2c0ca3b10c357650602b92596209b7492802f3768a6285f  libpsl-0.21.2.tar.gz
>  
>  # Hashes for license files:
> -sha256  7903413e7aa0bbcb77aa19150a3f1034edc0a760cff174f9a53efe4de6f056a4  COPYING
> +sha256  776cd11d54f0ea3e5efdd25ff1a0d952a1744215bb5858ea7aeb91808e70c9f7  COPYING
>  sha256  1d7f52747a9169751cdf2641a8299c0098e9ecdf9429296ffd55bdb14c9ed5b3  src/LICENSE.chromium
> diff --git a/package/libpsl/libpsl.mk b/package/libpsl/libpsl.mk
> index 742117e8ea..ef3c524b4f 100644
> --- a/package/libpsl/libpsl.mk
> +++ b/package/libpsl/libpsl.mk
> @@ -4,10 +4,11 @@
>  #
>  ################################################################################
>  
> -LIBPSL_VERSION = 0.21.1
> +LIBPSL_VERSION = 0.21.2
>  LIBPSL_SITE = https://github.com/rockdaboot/libpsl/releases/download/$(LIBPSL_VERSION)
>  LIBPSL_LICENSE = MIT, BSD-3-Clause
>  LIBPSL_LICENSE_FILES = COPYING src/LICENSE.chromium
> +LIBPSL_CONF_OPTS = -Dbuiltin=true
>  LIBPSL_DEPENDENCIES = host-pkgconf
>  LIBPSL_INSTALL_STAGING = YES
>  
> @@ -17,13 +18,13 @@ endif
>  
>  # The order of checks is the same as done by libpsl when configured.
>  ifeq ($(BR2_PACKAGE_LIBIDN2)$(BR2_PACKAGE_LIBUNISTRING),yy)
> -LIBPSL_CONF_OPTS += -Druntime=libidn2 -Dbuiltin=libidn2
> +LIBPSL_CONF_OPTS += -Druntime=libidn2
>  LIBPSL_DEPENDENCIES += libidn2 libunistring
>  else ifeq ($(BR2_PACKAGE_ICU),y)
> -LIBPSL_CONF_OPTS += -Druntime=libicu -Dbuiltin=libicu
> +LIBPSL_CONF_OPTS += -Druntime=libicu
>  LIBPSL_DEPENDENCIES += icu
>  else
> -LIBPSL_CONF_OPTS += -Druntime=libidn -Dbuiltin=libidn
> +LIBPSL_CONF_OPTS += -Druntime=libidn
>  LIBPSL_DEPENDENCIES += libidn libunistring
>  endif
>  
> -- 
> 2.39.0
> 
> 

Cheers,
—Adrián
Peter Korsgaard Jan. 27, 2023, 12:44 p.m. UTC | #2
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > - Drop patch (already in version)
 > - Update hash of COPYING (update in year:
 >   https://github.com/rockdaboot/libpsl/commit/927b202c69bcd9be4c30ba51c74925f2575e1a69)
 > - builtin is a boolean since
 >   https://github.com/rockdaboot/libpsl/commit/aa4909766c24c17bd1d9000ca419f6dc6b32c238

 > https://github.com/rockdaboot/libpsl/releases/tag/0.21.2

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed, thanks.
diff mbox series

Patch

diff --git a/package/libpsl/0001-Add-libiconv-dep-to-meson-build-again.patch b/package/libpsl/0001-Add-libiconv-dep-to-meson-build-again.patch
deleted file mode 100644
index 72f50aa593..0000000000
--- a/package/libpsl/0001-Add-libiconv-dep-to-meson-build-again.patch
+++ /dev/null
@@ -1,60 +0,0 @@ 
-From 5b63816e82287808e81b8603a528af3cc6fab354 Mon Sep 17 00:00:00 2001
-From: Zhao Zhili <zhilizhao@tencent.com>
-Date: Thu, 7 Jan 2021 20:59:17 +0800
-Subject: [PATCH] Add libiconv dep to meson build again
-
-Build success on macOS and ubuntu.
-
-[Retrieved from:
-https://github.com/rockdaboot/libpsl/commit/5b63816e82287808e81b8603a528af3cc6fab354]
-Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
----
- meson.build     | 14 ++++++++++++++
- src/meson.build |  2 +-
- 2 files changed, 15 insertions(+), 1 deletion(-)
-
-diff --git a/meson.build b/meson.build
-index f248171..fb2de31 100644
---- a/meson.build
-+++ b/meson.build
-@@ -20,6 +20,7 @@ libicu_dep = notfound
- libidn_dep = notfound
- libunistring = notfound
- networking_deps = notfound
-+libiconv_dep = notfound
- 
- # FIXME: Cleanup this when Meson gets 'feature-combo':
- # https://github.com/mesonbuild/meson/issues/4566
-@@ -86,6 +87,19 @@ endif
- if libidn2_dep.found() or libidn_dep.found()
-   # Check for libunistring, we need it for psl_str_to_utf8lower()
-   libunistring = cc.find_library('unistring')
-+  found_iconv = false
-+  if cc.has_function('iconv_open')
-+    libiconv_dep = []
-+    found_iconv = true
-+  endif
-+  if not found_iconv and cc.has_header_symbol('iconv.h', 'iconv_open')
-+    libiconv_dep = [cc.find_library('iconv')]
-+    found_iconv = true
-+  endif
-+
-+  if not found_iconv
-+    error('iconv implementation not found')
-+  endif
- endif
- 
- if host_machine.system() == 'windows'
-diff --git a/src/meson.build b/src/meson.build
-index 0d1dcdf..746bf44 100644
---- a/src/meson.build
-+++ b/src/meson.build
-@@ -19,7 +19,7 @@ cargs = [
- libpsl = library('psl', sources, suffixes_dafsa_h,
-   include_directories : [configinc, includedir],
-   c_args : cargs,
--  dependencies : [libidn2_dep, libidn_dep, libicu_dep, libunistring, networking_deps],
-+  dependencies : [libidn2_dep, libidn_dep, libicu_dep, libunistring, networking_deps, libiconv_dep],
-   version: lt_version,
-   install: true,
- )
diff --git a/package/libpsl/libpsl.hash b/package/libpsl/libpsl.hash
index 552935e9ec..a35a609ff2 100644
--- a/package/libpsl/libpsl.hash
+++ b/package/libpsl/libpsl.hash
@@ -1,6 +1,6 @@ 
 # Locally generated
-sha256  ac6ce1e1fbd4d0254c4ddb9d37f1fa99dec83619c1253328155206b896210d4c  libpsl-0.21.1.tar.gz
+sha256  e35991b6e17001afa2c0ca3b10c357650602b92596209b7492802f3768a6285f  libpsl-0.21.2.tar.gz
 
 # Hashes for license files:
-sha256  7903413e7aa0bbcb77aa19150a3f1034edc0a760cff174f9a53efe4de6f056a4  COPYING
+sha256  776cd11d54f0ea3e5efdd25ff1a0d952a1744215bb5858ea7aeb91808e70c9f7  COPYING
 sha256  1d7f52747a9169751cdf2641a8299c0098e9ecdf9429296ffd55bdb14c9ed5b3  src/LICENSE.chromium
diff --git a/package/libpsl/libpsl.mk b/package/libpsl/libpsl.mk
index 742117e8ea..ef3c524b4f 100644
--- a/package/libpsl/libpsl.mk
+++ b/package/libpsl/libpsl.mk
@@ -4,10 +4,11 @@ 
 #
 ################################################################################
 
-LIBPSL_VERSION = 0.21.1
+LIBPSL_VERSION = 0.21.2
 LIBPSL_SITE = https://github.com/rockdaboot/libpsl/releases/download/$(LIBPSL_VERSION)
 LIBPSL_LICENSE = MIT, BSD-3-Clause
 LIBPSL_LICENSE_FILES = COPYING src/LICENSE.chromium
+LIBPSL_CONF_OPTS = -Dbuiltin=true
 LIBPSL_DEPENDENCIES = host-pkgconf
 LIBPSL_INSTALL_STAGING = YES
 
@@ -17,13 +18,13 @@  endif
 
 # The order of checks is the same as done by libpsl when configured.
 ifeq ($(BR2_PACKAGE_LIBIDN2)$(BR2_PACKAGE_LIBUNISTRING),yy)
-LIBPSL_CONF_OPTS += -Druntime=libidn2 -Dbuiltin=libidn2
+LIBPSL_CONF_OPTS += -Druntime=libidn2
 LIBPSL_DEPENDENCIES += libidn2 libunistring
 else ifeq ($(BR2_PACKAGE_ICU),y)
-LIBPSL_CONF_OPTS += -Druntime=libicu -Dbuiltin=libicu
+LIBPSL_CONF_OPTS += -Druntime=libicu
 LIBPSL_DEPENDENCIES += icu
 else
-LIBPSL_CONF_OPTS += -Druntime=libidn -Dbuiltin=libidn
+LIBPSL_CONF_OPTS += -Druntime=libidn
 LIBPSL_DEPENDENCIES += libidn libunistring
 endif