[Buildroot] [PATCH 1/1] package/libnfc: support pcsc-lite

Thomas Petazzoni thomas.petazzoni at bootlin.com
Sat Jun 1 12:25:39 UTC 2019


Hello Raul,

On Mon, 27 May 2019 11:15:50 +0200
Thomas Petazzoni <thomas.petazzoni at bootlin.com> wrote:

> Here as well: please send with git send-email so that we can do a
> proper review.
> 
> A few comments though:
> 
>  - We need you to use your real name as the patch author and for the
>    Signed-off-by line
> 
>  - Why are you making pcsc-lite a mandatory dependency of libnfc ?
>    Since libnfc builds fine today with pcsc-lite, it means it should be
>    an optional dependency. Either add a sub-option to enable the
>    different libnfc drivers, or automatically add pcsc-lite to the
>    dependencies if BR2_PACKAGE_PCSC_LITE=y.

I just wanted to make sure that you have seen my comments on this
patch. I'm marking it as "Changes Requested" in our patch tracking
system. Therefore, if you don't send an updated version of the patch,
we will forget about it.

Thanks!

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


More information about the buildroot mailing list