All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v3 1/1] package/opensc: new package
@ 2020-10-15 15:01 Yair Ben-Avraham
  2020-10-15 21:06 ` Thomas Petazzoni
  0 siblings, 1 reply; 3+ messages in thread
From: Yair Ben-Avraham @ 2020-10-15 15:01 UTC (permalink / raw)
  To: buildroot

Open source smart card tools and middleware.
OpenSC provides a set of libraries and utilities to work with smart cards. Its
main focus is on cards that support cryptographic operations, and facilitate
their use in security applications such as authentication, mail encryption and
digital signatures. OpenSC implements the standard APIs to smart cards, e.g.
PKCS#11 API, Windows? Smart Card Minidriver and macOS CryptoTokenKit.

Signed-off-by: Yair Ben-Avraham <yairba@protonmail.com>
---
 package/Config.in          |  1 +
 package/opensc/Config.in   | 23 +++++++++++++++++++++++
 package/opensc/opensc.hash |  3 +++
 package/opensc/opensc.mk   | 14 ++++++++++++++
 4 files changed, 41 insertions(+)
 create mode 100644 package/opensc/Config.in
 create mode 100644 package/opensc/opensc.hash
 create mode 100644 package/opensc/opensc.mk

diff --git a/package/Config.in b/package/Config.in
index 09a332e3b9..1a697352c9 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1353,6 +1353,7 @@ menu "Crypto"
 	source "package/libuecc/Config.in"
 	source "package/mbedtls/Config.in"
 	source "package/nettle/Config.in"
+	source "package/opensc/Config.in"
 	source "package/openssl/Config.in"
 	source "package/pkcs11-helper/Config.in"
 	source "package/rhash/Config.in"
diff --git a/package/opensc/Config.in b/package/opensc/Config.in
new file mode 100644
index 0000000000..973c50caec
--- /dev/null
+++ b/package/opensc/Config.in
@@ -0,0 +1,23 @@
+config BR2_PACKAGE_OPENSC
+	bool "opensc"
+	depends on BR2_TOOLCHAIN_HAS_THREADS # pcsc-lite, libusb
+	depends on BR2_USE_MMU # pcsc-lite
+	depends on BR2_USE_WCHAR # wprintf
+	depends on !BR2_STATIC_LIBS # pcsc-lite
+	select BR2_PACKAGE_LIBUSB
+	select BR2_PACKAGE_PCSC_LITE
+	help
+	  Open source smart card tools and middleware.
+	  OpenSC provides a set of libraries and utilities to work with smart
+	  cards. Its main focus is on cards that support cryptographic
+	  operations, and facilitate their use in security applications such as
+	  authentication, mail encryption and digital signatures. OpenSC
+	  implements the standard APIs to smart cards, e.g. PKCS#11 API,
+	  Windows? Smart Card Minidriver and macOS CryptoTokenKit.
+
+	  https://github.com/OpenSC/OpenSC
+
+comment "opensc needs a toolchain w/ wchar threads, dynamic library"
+	depends on BR2_USE_MMU
+	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_USE_WCHAR \
+		|| BR2_STATIC_LIBS
diff --git a/package/opensc/opensc.hash b/package/opensc/opensc.hash
new file mode 100644
index 0000000000..aceb51e2de
--- /dev/null
+++ b/package/opensc/opensc.hash
@@ -0,0 +1,3 @@
+# Locally computed:
+sha256  bbf4b4f4a44463645c90a525e820a8059b2f742a53b7b944f941de3c97ba4863  opensc-0.20.0.tar.gz
+sha256  a190dc9c8043755d90f8b0a75fa66b9e42d4af4c980bf5ddc633f0124db3cee7  COPYING
diff --git a/package/opensc/opensc.mk b/package/opensc/opensc.mk
new file mode 100644
index 0000000000..8bd6fceece
--- /dev/null
+++ b/package/opensc/opensc.mk
@@ -0,0 +1,14 @@
+################################################################################
+#
+# opensc
+#
+################################################################################
+
+OPENSC_VERSION = 0.20.0
+OPENSC_SITE = https://github.com/OpenSC/OpenSC/releases/download/$(OPENSC_VERSION)
+OPENSC_LICENSE = LGPL-2.1+
+OPENSC_LICENSE_FILES = COPYING
+OPENSC_INSTALL_STAGING = YES
+OPENSC_DEPENDENCIES = libusb pcsc-lite
+
+$(eval $(autotools-package))
--
2.20.1

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

* [Buildroot] [PATCH v3 1/1] package/opensc: new package
  2020-10-15 15:01 [Buildroot] [PATCH v3 1/1] package/opensc: new package Yair Ben-Avraham
@ 2020-10-15 21:06 ` Thomas Petazzoni
  2020-10-18  6:07   ` Yair Ben-Avraham
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni @ 2020-10-15 21:06 UTC (permalink / raw)
  To: buildroot

Hello,

On Thu, 15 Oct 2020 15:01:36 +0000
Yair Ben-Avraham via buildroot <buildroot@busybox.net> wrote:

> Open source smart card tools and middleware.
> OpenSC provides a set of libraries and utilities to work with smart cards. Its
> main focus is on cards that support cryptographic operations, and facilitate
> their use in security applications such as authentication, mail encryption and
> digital signatures. OpenSC implements the standard APIs to smart cards, e.g.
> PKCS#11 API, Windows? Smart Card Minidriver and macOS CryptoTokenKit.
> 
> Signed-off-by: Yair Ben-Avraham <yairba@protonmail.com>

I was about to apply, but unfortunately the build fails with the following defconfig:

BR2_arm=y
BR2_TOOLCHAIN_EXTERNAL=y
BR2_TOOLCHAIN_EXTERNAL_CUSTOM=y
BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD=y
BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/br-arm-full-2020.02.tar.bz2"
BR2_TOOLCHAIN_EXTERNAL_GCC_5=y
BR2_TOOLCHAIN_EXTERNAL_HEADERS_4_14=y
BR2_TOOLCHAIN_EXTERNAL_LOCALE=y
# BR2_TOOLCHAIN_EXTERNAL_HAS_THREADS_DEBUG is not set
BR2_TOOLCHAIN_EXTERNAL_CXX=y
BR2_INIT_NONE=y
BR2_SYSTEM_BIN_SH_NONE=y
# BR2_PACKAGE_BUSYBOX is not set
BR2_PACKAGE_OPENSC=y
# BR2_TARGET_ROOTFS_TAR is not set

The failure is:

pkcs15-asepcos.c: In function ?asepcos_do_store_pin.isra.3?:
pkcs15-asepcos.c:307:33: error: ?akn? may be used uninitialized in this function [-Werror=maybe-uninitialized]
  auth_info->attrs.pin.reference = akn;
                                 ^
cc1: all warnings being treated as errors

Since you're going to send a new iteration for this, could you also
make sure to update the DEVELOPERS file in the same patch to add
yourself as a maintainer for this package ?

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [Buildroot] [PATCH v3 1/1] package/opensc: new package
  2020-10-15 21:06 ` Thomas Petazzoni
@ 2020-10-18  6:07   ` Yair Ben-Avraham
  0 siblings, 0 replies; 3+ messages in thread
From: Yair Ben-Avraham @ 2020-10-18  6:07 UTC (permalink / raw)
  To: buildroot

Fixed in v4 for this patch.

In general, can you publish/refer to a minimal defconfigs for testing?

Thanks,
Yair

??????? Original Message ???????
On Thursday, October 15, 2020 11:06 PM, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:

> Hello,
>
> On Thu, 15 Oct 2020 15:01:36 +0000
> Yair Ben-Avraham via buildroot buildroot at busybox.net wrote:
>
> > Open source smart card tools and middleware.
> > OpenSC provides a set of libraries and utilities to work with smart cards. Its
> > main focus is on cards that support cryptographic operations, and facilitate
> > their use in security applications such as authentication, mail encryption and
> > digital signatures. OpenSC implements the standard APIs to smart cards, e.g.
> > PKCS#11 API, Windows? Smart Card Minidriver and macOS CryptoTokenKit.
> > Signed-off-by: Yair Ben-Avraham yairba at protonmail.com
>
> I was about to apply, but unfortunately the build fails with the following defconfig:
>
> BR2_arm=y
> BR2_TOOLCHAIN_EXTERNAL=y
> BR2_TOOLCHAIN_EXTERNAL_CUSTOM=y
> BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD=y
> BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/br-arm-full-2020.02.tar.bz2"
> BR2_TOOLCHAIN_EXTERNAL_GCC_5=y
> BR2_TOOLCHAIN_EXTERNAL_HEADERS_4_14=y
> BR2_TOOLCHAIN_EXTERNAL_LOCALE=y
>
> BR2_TOOLCHAIN_EXTERNAL_HAS_THREADS_DEBUG is not set
>
> ====================================================
>
> BR2_TOOLCHAIN_EXTERNAL_CXX=y
> BR2_INIT_NONE=y
> BR2_SYSTEM_BIN_SH_NONE=y
>
> BR2_PACKAGE_BUSYBOX is not set
>
> ===============================
>
> BR2_PACKAGE_OPENSC=y
>
> BR2_TARGET_ROOTFS_TAR is not set
>
> =================================
>
> The failure is:
>
> pkcs15-asepcos.c: In function ?asepcos_do_store_pin.isra.3?:
> pkcs15-asepcos.c:307:33: error: ?akn? may be used uninitialized in this function [-Werror=maybe-uninitialized]
> auth_info->attrs.pin.reference = akn;
>
>                                  ^
>
>
> cc1: all warnings being treated as errors
>
> Since you're going to send a new iteration for this, could you also
> make sure to update the DEVELOPERS file in the same patch to add
> yourself as a maintainer for this package ?
>
> Thanks a lot!
>
> Thomas
>
> ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com

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

end of thread, other threads:[~2020-10-18  6:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-15 15:01 [Buildroot] [PATCH v3 1/1] package/opensc: new package Yair Ben-Avraham
2020-10-15 21:06 ` Thomas Petazzoni
2020-10-18  6:07   ` Yair Ben-Avraham

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.