Message ID | 20240423062020.2543618-1-zenichi.amano@gahojin.co.jp |
---|---|
State | Changes Requested |
Headers | show |
Series | [1/1] package/pcsc-lite: bump version to 2.1.0 | expand |
Hello Zenichi, Thanks a lot for your patch! I have a request below. On Tue, 23 Apr 2024 15:20:20 +0900 Zenichi Amano <zenichi.amano@gahojin.co.jp> wrote: > +ifneq ($(BR2_PACKAGE_POLKIT),y) > +PCSC_LITE_CONF_OPTS += --disable-polkit > +endif We really like to have both sides of the conditions explicitly, i.e something like: ifeq ($(BR2_PACKAGE_POLKIT),y) PCSC_LITE_CONF_OPTS += --enable-polkit PCSC_LITE_DEPENDENCIES += polkit else PCSC_LITE_CONF_OPTS += --disable-polkit endif or, if you're not interested in testing the polkit support, you can also just unconditionally pass --disable-polkit, and let someone else interested by polkit support look into this topic. Indeed, your change is incorrect, as BR2_PACKAGE_POLKIT=y implies that polkit is enabled, but not that it has been built before. So with your change, it is possible that pcsc-lite gets built before polkit... and you don't pass --disable-polkit in this case. Could you have a look at providing an updated patch? Thanks a lot, Thomas
diff --git a/package/pcsc-lite/pcsc-lite.hash b/package/pcsc-lite/pcsc-lite.hash index e0face9ddb..806792faee 100644 --- a/package/pcsc-lite/pcsc-lite.hash +++ b/package/pcsc-lite/pcsc-lite.hash @@ -1,4 +1,4 @@ # Locally calculated -sha256 d6c3e2b64510e5ed6fcd3323febf2cc2a8e5fda5a6588c7671f2d77f9f189356 pcsc-lite-2.0.0.tar.bz2 +sha256 85cab61cc744c81e2bc432656863293b8428d0136f079e3b12a84b335b5b35aa pcsc-lite-2.1.0.tar.bz2 sha256 40d4358bb8dc8d30f5365ae6d999b6f820bbc051b292912482f2e4d0ebd837e1 COPYING sha256 8ceb4b9ee5adedde47b31e975c1d90c73ad27b6b165a1dcd80c7c545eb65b903 GPL-3.0.txt diff --git a/package/pcsc-lite/pcsc-lite.mk b/package/pcsc-lite/pcsc-lite.mk index 2b2114835e..e873c23bd0 100644 --- a/package/pcsc-lite/pcsc-lite.mk +++ b/package/pcsc-lite/pcsc-lite.mk @@ -4,7 +4,7 @@ # ################################################################################ -PCSC_LITE_VERSION = 2.0.0 +PCSC_LITE_VERSION = 2.1.0 PCSC_LITE_SOURCE = pcsc-lite-$(PCSC_LITE_VERSION).tar.bz2 PCSC_LITE_SITE = https://pcsclite.apdu.fr/files PCSC_LITE_INSTALL_STAGING = YES @@ -45,6 +45,10 @@ ifeq ($(BR2_PACKAGE_PCSC_LITE_EMBEDDED),y) PCSC_LITE_CONF_OPTS += --enable-embedded endif +ifneq ($(BR2_PACKAGE_POLKIT),y) +PCSC_LITE_CONF_OPTS += --disable-polkit +endif + ifeq ($(BR2_TOOLCHAIN_HAS_LIBATOMIC),y) PCSC_LITE_CONF_ENV += LIBS=-latomic endif
add configuration to disable polkit if it is not available since polkit is enabled by default in 2.0.1. https://github.com/LudovicRousseau/PCSC/releases/tag/2.1.0 https://github.com/LudovicRousseau/PCSC/releases/tag/2.0.3 https://github.com/LudovicRousseau/PCSC/releases/tag/2.0.2 https://github.com/LudovicRousseau/PCSC/releases/tag/2.0.1 Signed-off-by: Zenichi Amano <zenichi.amano@gahojin.co.jp> --- package/pcsc-lite/pcsc-lite.hash | 2 +- package/pcsc-lite/pcsc-lite.mk | 6 +++++- 2 files changed, 6 insertions(+), 2 deletions(-)