[Buildroot] [PATCH v4, 1/1] package/uacme: don't allow mbedtls with ualpn

[Fabrice Fontaine]

Le dim. 26 avr. 2020 à 14:13, Thomas Petazzoni
<thomas.petazzoni at bootlin.com> a écrit :
>
> On Sun, 26 Apr 2020 13:36:39 +0200
> "Yann E. MORIN" <yann.morin.1998 at free.fr> wrote:
>
> > > +choice
> > > +   prompt "Crypto Backend"
> > > +   help
> > > +     Select crypto library to be used in uacme.
> > > +
> > > +config BR2_PACKAGE_UACME_GNUTLS
> > > +   bool "gnutls"
> > > +   depends on BR2_PACKAGE_GNUTLS
> > > +
> > > +config BR2_PACKAGE_UACME_MBEDTLS
> > > +   bool "mbedtls"
> > > +   depends on BR2_PACKAGE_MBEDTLS
> > > +   depends on !BR2_PACKAGE_UACME_UALPN
> > > +
> > > +comment "mbedtls crypto backend unavailable with ualpn"
> > > +   depends on BR2_PACKAGE_MBEDTLS
> > > +   depends on BR2_PACKAGE_UACME_UALPN
> > > +
> > > +config BR2_PACKAGE_UACME_OPENSSL
> > > +   bool "openssl"
> > > +   depends on BR2_PACKAGE_OPENSSL
> > > +
> > > +endchoice
> >
> > Sorry, but this is still not correct: enable mbedtls, then enable uacme
> > and ualpn: there is no crypto backend selectable in the choice...
>
> Do we need a choice in the first place ?
v2 didn't add a choice but it was a bit too confusing so I made v3/v4
upon Yann's request. I'll wait for your feedback before making a new
iteration.
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
Best Regards,

Fabrice