[Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency

[Fabrice Fontaine]

Hi all,

Le mer. 3 nov. 2021 à 21:20, James Hilliard
<james.hilliard1 at gmail.com> a écrit :
>
> On Wed, Nov 3, 2021 at 2:12 PM Peter Seiderer <ps.report at gmx.net> wrote:
> >
> > Hello James,
> >
> > On Wed, 3 Nov 2021 13:51:09 -0600, James Hilliard <james.hilliard1 at gmail.com> wrote:
> >
> > > On Wed, Nov 3, 2021 at 1:48 PM Peter Seiderer <ps.report at gmx.net> wrote:
> > > >
> > > > Hello James, Fabrice, *
> > > >
> > > > On Fri, 29 Oct 2021 00:59:57 -0600, James Hilliard <james.hilliard1 at gmail.com> wrote:
> > > >
> > > > > This dependency wasn't added during the removal of
> > > > > python3-cryptography.
> > > >
> > > > The conversion/dropping of python3-cryptography was done with commits [1] and [2],
> > > > and the commit log of [1] states the dropping of the host-python3-six dependency
> > > > was intentionally:
> > > >
> > > >         [...]
> > > >         While at it, drop host-python3-pip and host-python3-six as these
> > > >         dependencies are not needed. Indeed, pip was never a dependency of
> > > >         cryptography and six is not a dependency with python3 and will be
> > > >         dropped in version 3.4 and
> > > >         https://github.com/pyca/cryptography/commit/e66db8079d3fbd0110e87ece1fd48f4bfd9e48b9
> > > >         [...]
> > > >
> > > > But the package python-cryptography is still at version 3.3.2 (and clearly depends on six)
> > > >
> > > > @Fabrice: version bump pending?
> > >
> > > I actually have a patch from a while back bumping the version...but it
> > > also requires rust
> > > support and that would probably cause a bit of breakage:
> > > https://patchwork.ozlabs.org/project/buildroot/patch/20210318044253.3202355-2-james.hilliard1@gmail.com/
> > >
> > > So it's probably going to be a while before we can update cryptography
> > > if I had to guess.
> >
> > Clear vote for keeping the old version and re-adding the host-python3-six
> > dependency ;-)
> >
> > Mind to re-spin your patch with some explanation?
>
> Sure, I mostly just copied your explanation of the issue:
> https://patchwork.ozlabs.org/project/buildroot/patch/20211103201932.296897-1-james.hilliard1@gmail.com/
I already sent a patch a few days ago:
https://patchwork.ozlabs.org/project/buildroot/patch/20211026163222.3912644-1-fontaine.fabrice@gmail.com/
>
> >
> > Regards,
> > Peter
> >
> > >
> > > >
> > > > @James: with a bit of explanation added to the commit log you can add my
> > > >
> > > > Reviewed-by: Peter Seiderer <ps.report at gmx.net>
> > > >
> > > > But although fine with an version bump of the package ;-)
> > > >
> > > > Regards,
> > > > Peter
> > > >
> > > > [1] https://git.buildroot.net/buildroot/commit/?id=95a63a34ac9204c4fe069154783bd97ed48c3ae2
> > > > [2] https://git.buildroot.net/buildroot/commit/?id=43147827a7b759248288fad1d97130c027c23e9a
> > > >
> > > > >
> > > > > Fixes:
> > > > >  - http://autobuild.buildroot.net/results/971/9711ccebb9c3e2fa8f17d969d8b431d4301677e3/build-end.log
> > > > >
> > > > > Signed-off-by: James Hilliard <james.hilliard1 at gmail.com>
> > > > > ---
> > > > >  package/python-cryptography/python-cryptography.mk | 5 ++++-
> > > > >  1 file changed, 4 insertions(+), 1 deletion(-)
> > > > >
> > > > > diff --git a/package/python-cryptography/python-cryptography.mk b/package/python-cryptography/python-cryptography.mk
> > > > > index e21b9db042..081dfc06a0 100644
> > > > > --- a/package/python-cryptography/python-cryptography.mk
> > > > > +++ b/package/python-cryptography/python-cryptography.mk
> > > > > @@ -14,7 +14,10 @@ PYTHON_CRYPTOGRAPHY_CPE_ID_VENDOR = cryptography_project
> > > > >  PYTHON_CRYPTOGRAPHY_CPE_ID_PRODUCT = cryptography
> > > > >  PYTHON_CRYPTOGRAPHY_DEPENDENCIES = host-python-cffi openssl
> > > > >  HOST_PYTHON_CRYPTOGRAPHY_NEEDS_HOST_PYTHON = python3
> > > > > -HOST_PYTHON_CRYPTOGRAPHY_DEPENDENCIES = host-python3-cffi host-openssl
> > > > > +HOST_PYTHON_CRYPTOGRAPHY_DEPENDENCIES = \
> > > > > +     host-python3-cffi \
> > > > > +     host-python3-six \
> > > > > +     host-openssl
> > > > >
> > > > >  $(eval $(python-package))
> > > > >  $(eval $(host-python-package))
> > > >
> > > _______________________________________________
> > > buildroot mailing list
> > > buildroot at buildroot.org
> > > https://lists.buildroot.org/mailman/listinfo/buildroot
> >
Best Regards,

Fabrice