All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency
@ 2021-10-29  6:59 James Hilliard
  2021-11-03 19:48 ` Peter Seiderer
  0 siblings, 1 reply; 7+ messages in thread
From: James Hilliard @ 2021-10-29  6:59 UTC (permalink / raw)
  To: buildroot; +Cc: James Hilliard, Asaf Kahlon, Thomas Petazzoni

This dependency wasn't added during the removal of
python3-cryptography.

Fixes:
 - http://autobuild.buildroot.net/results/971/9711ccebb9c3e2fa8f17d969d8b431d4301677e3/build-end.log

Signed-off-by: James Hilliard <james.hilliard1@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))
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency
  2021-10-29  6:59 [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency James Hilliard
@ 2021-11-03 19:48 ` Peter Seiderer
  2021-11-03 19:51   ` James Hilliard
  0 siblings, 1 reply; 7+ messages in thread
From: Peter Seiderer @ 2021-11-03 19:48 UTC (permalink / raw)
  To: James Hilliard; +Cc: Fabrice Fontaine, Thomas Petazzoni, Asaf Kahlon, buildroot

Hello James, Fabrice, *

On Fri, 29 Oct 2021 00:59:57 -0600, James Hilliard <james.hilliard1@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?

@James: with a bit of explanation added to the commit log you can add my

Reviewed-by: Peter Seiderer <ps.report@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@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@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency
  2021-11-03 19:48 ` Peter Seiderer
@ 2021-11-03 19:51   ` James Hilliard
  2021-11-03 20:12     ` Peter Seiderer
  0 siblings, 1 reply; 7+ messages in thread
From: James Hilliard @ 2021-11-03 19:51 UTC (permalink / raw)
  To: Peter Seiderer; +Cc: Fabrice Fontaine, Thomas Petazzoni, Asaf Kahlon, buildroot

On Wed, Nov 3, 2021 at 1:48 PM Peter Seiderer <ps.report@gmx.net> wrote:
>
> Hello James, Fabrice, *
>
> On Fri, 29 Oct 2021 00:59:57 -0600, James Hilliard <james.hilliard1@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.

>
> @James: with a bit of explanation added to the commit log you can add my
>
> Reviewed-by: Peter Seiderer <ps.report@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@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@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency
  2021-11-03 19:51   ` James Hilliard
@ 2021-11-03 20:12     ` Peter Seiderer
  2021-11-03 20:20       ` James Hilliard
  0 siblings, 1 reply; 7+ messages in thread
From: Peter Seiderer @ 2021-11-03 20:12 UTC (permalink / raw)
  To: James Hilliard; +Cc: Asaf Kahlon, Fabrice Fontaine, Thomas Petazzoni, buildroot

Hello James,

On Wed, 3 Nov 2021 13:51:09 -0600, James Hilliard <james.hilliard1@gmail.com> wrote:

> On Wed, Nov 3, 2021 at 1:48 PM Peter Seiderer <ps.report@gmx.net> wrote:
> >
> > Hello James, Fabrice, *
> >
> > On Fri, 29 Oct 2021 00:59:57 -0600, James Hilliard <james.hilliard1@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?

Regards,
Peter

>
> >
> > @James: with a bit of explanation added to the commit log you can add my
> >
> > Reviewed-by: Peter Seiderer <ps.report@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@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@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency
  2021-11-03 20:12     ` Peter Seiderer
@ 2021-11-03 20:20       ` James Hilliard
  2021-11-03 22:37         ` Fabrice Fontaine
  0 siblings, 1 reply; 7+ messages in thread
From: James Hilliard @ 2021-11-03 20:20 UTC (permalink / raw)
  To: Peter Seiderer; +Cc: Asaf Kahlon, Fabrice Fontaine, Thomas Petazzoni, buildroot

On Wed, Nov 3, 2021 at 2:12 PM Peter Seiderer <ps.report@gmx.net> wrote:
>
> Hello James,
>
> On Wed, 3 Nov 2021 13:51:09 -0600, James Hilliard <james.hilliard1@gmail.com> wrote:
>
> > On Wed, Nov 3, 2021 at 1:48 PM Peter Seiderer <ps.report@gmx.net> wrote:
> > >
> > > Hello James, Fabrice, *
> > >
> > > On Fri, 29 Oct 2021 00:59:57 -0600, James Hilliard <james.hilliard1@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/

>
> Regards,
> Peter
>
> >
> > >
> > > @James: with a bit of explanation added to the commit log you can add my
> > >
> > > Reviewed-by: Peter Seiderer <ps.report@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@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@buildroot.org
> > https://lists.buildroot.org/mailman/listinfo/buildroot
>
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency
  2021-11-03 20:20       ` James Hilliard
@ 2021-11-03 22:37         ` Fabrice Fontaine
  2021-11-04 17:30           ` Peter Seiderer
  0 siblings, 1 reply; 7+ messages in thread
From: Fabrice Fontaine @ 2021-11-03 22:37 UTC (permalink / raw)
  To: James Hilliard; +Cc: Peter Seiderer, Asaf Kahlon, Thomas Petazzoni, buildroot

Hi all,

Le mer. 3 nov. 2021 à 21:20, James Hilliard
<james.hilliard1@gmail.com> a écrit :
>
> On Wed, Nov 3, 2021 at 2:12 PM Peter Seiderer <ps.report@gmx.net> wrote:
> >
> > Hello James,
> >
> > On Wed, 3 Nov 2021 13:51:09 -0600, James Hilliard <james.hilliard1@gmail.com> wrote:
> >
> > > On Wed, Nov 3, 2021 at 1:48 PM Peter Seiderer <ps.report@gmx.net> wrote:
> > > >
> > > > Hello James, Fabrice, *
> > > >
> > > > On Fri, 29 Oct 2021 00:59:57 -0600, James Hilliard <james.hilliard1@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@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@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@buildroot.org
> > > https://lists.buildroot.org/mailman/listinfo/buildroot
> >
Best Regards,

Fabrice
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency
  2021-11-03 22:37         ` Fabrice Fontaine
@ 2021-11-04 17:30           ` Peter Seiderer
  0 siblings, 0 replies; 7+ messages in thread
From: Peter Seiderer @ 2021-11-04 17:30 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: James Hilliard, Asaf Kahlon, Thomas Petazzoni, buildroot

Hello Fabrice,

On Wed, 3 Nov 2021 23:37:17 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> Hi all,
> 
> Le mer. 3 nov. 2021 à 21:20, James Hilliard
> <james.hilliard1@gmail.com> a écrit :
> >
> > On Wed, Nov 3, 2021 at 2:12 PM Peter Seiderer <ps.report@gmx.net> wrote:  
> > >
> > > Hello James,
> > >
> > > On Wed, 3 Nov 2021 13:51:09 -0600, James Hilliard <james.hilliard1@gmail.com> wrote:
> > >  
> > > > On Wed, Nov 3, 2021 at 1:48 PM Peter Seiderer <ps.report@gmx.net> wrote:  
> > > > >
> > > > > Hello James, Fabrice, *
> > > > >
> > > > > On Fri, 29 Oct 2021 00:59:57 -0600, James Hilliard <james.hilliard1@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/

Sorry for missing this one...

Regards,
Peter

> >  
> > >
> > > Regards,
> > > Peter
> > >  
> > > >  
> > > > >
> > > > > @James: with a bit of explanation added to the commit log you can add my
> > > > >
> > > > > Reviewed-by: Peter Seiderer <ps.report@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@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@buildroot.org
> > > > https://lists.buildroot.org/mailman/listinfo/buildroot  
> > >  
> Best Regards,
> 
> Fabrice

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

end of thread, other threads:[~2021-11-04 17:30 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-29  6:59 [Buildroot] [PATCH 1/1] package/python-cryptography: add host-python3-six host dependency James Hilliard
2021-11-03 19:48 ` Peter Seiderer
2021-11-03 19:51   ` James Hilliard
2021-11-03 20:12     ` Peter Seiderer
2021-11-03 20:20       ` James Hilliard
2021-11-03 22:37         ` Fabrice Fontaine
2021-11-04 17:30           ` Peter Seiderer

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.