* [Buildroot] [PATCH 2/6] package/python-appdirs: add new package
2022-04-03 22:19 [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Emile Cormier
@ 2022-04-03 22:19 ` Emile Cormier
2022-08-07 14:50 ` Thomas Petazzoni via buildroot
2022-04-03 22:19 ` [Buildroot] [PATCH 3/6] package/python-zlmdb: " Emile Cormier
` (4 subsequent siblings)
5 siblings, 1 reply; 21+ messages in thread
From: Emile Cormier @ 2022-04-03 22:19 UTC (permalink / raw)
To: buildroot
Cc: Asaf Kahlon, James Hilliard, Thomas Petazzoni, Emile Cormier,
Mauro Condarelli
From: ecorm <ecorm@users.noreply.github.com>
Required by python-twisted[conch], which is required by python-crossbar.
Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
---
package/Config.in | 1 +
package/python-appdirs/Config.in | 7 +++++++
package/python-appdirs/python-appdirs.hash | 5 +++++
package/python-appdirs/python-appdirs.mk | 14 ++++++++++++++
4 files changed, 27 insertions(+)
create mode 100644 package/python-appdirs/Config.in
create mode 100644 package/python-appdirs/python-appdirs.hash
create mode 100644 package/python-appdirs/python-appdirs.mk
diff --git a/package/Config.in b/package/Config.in
index 0d5d763180..a1964df5f1 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -929,6 +929,7 @@ menu "External python modules"
source "package/python-aiozipkin/Config.in"
source "package/python-alsaaudio/Config.in"
source "package/python-ansicolors/Config.in"
+ source "package/python-appdirs/Config.in"
source "package/python-argh/Config.in"
source "package/python-argon2-cffi/Config.in"
source "package/python-argon2-cffi-bindings/Config.in"
diff --git a/package/python-appdirs/Config.in b/package/python-appdirs/Config.in
new file mode 100644
index 0000000000..5585759aa6
--- /dev/null
+++ b/package/python-appdirs/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_PYTHON_APPDIRS
+ bool "python-appdirs"
+ help
+ A small Python module for determining appropriate platform-
+ specific dirs, e.g. a "user data dir".
+
+ http://github.com/ActiveState/appdirs
diff --git a/package/python-appdirs/python-appdirs.hash b/package/python-appdirs/python-appdirs.hash
new file mode 100644
index 0000000000..52689b8b2c
--- /dev/null
+++ b/package/python-appdirs/python-appdirs.hash
@@ -0,0 +1,5 @@
+# md5, sha256 from https://pypi.org/pypi/appdirs/json
+md5 d6bca12613174185dd9abc8a29f4f012 appdirs-1.4.4.tar.gz
+sha256 7d5d0167b2b1ba821647616af46a749d1c653740dd0d2415100fe26e27afdf41 appdirs-1.4.4.tar.gz
+# Locally computed sha256 checksums
+sha256 36ddb4d0a745a93ab203203d7190814b1b89727d254caff48c7a7afbbd47000b LICENSE.txt
diff --git a/package/python-appdirs/python-appdirs.mk b/package/python-appdirs/python-appdirs.mk
new file mode 100644
index 0000000000..222aadbb29
--- /dev/null
+++ b/package/python-appdirs/python-appdirs.mk
@@ -0,0 +1,14 @@
+################################################################################
+#
+# python-appdirs
+#
+################################################################################
+
+PYTHON_APPDIRS_VERSION = 1.4.4
+PYTHON_APPDIRS_SOURCE = appdirs-$(PYTHON_APPDIRS_VERSION).tar.gz
+PYTHON_APPDIRS_SITE = https://files.pythonhosted.org/packages/d7/d8/05696357e0311f5b5c316d7b95f46c669dd9c15aaeecbb48c7d0aeb88c40
+PYTHON_APPDIRS_SETUP_TYPE = setuptools
+PYTHON_APPDIRS_LICENSE = MIT
+PYTHON_APPDIRS_LICENSE_FILES = LICENSE.txt
+
+$(eval $(python-package))
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [Buildroot] [PATCH 2/6] package/python-appdirs: add new package
2022-04-03 22:19 ` [Buildroot] [PATCH 2/6] package/python-appdirs: add new package Emile Cormier
@ 2022-08-07 14:50 ` Thomas Petazzoni via buildroot
0 siblings, 0 replies; 21+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-08-07 14:50 UTC (permalink / raw)
To: Emile Cormier; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
Hello Emile,
On Sun, 3 Apr 2022 19:19:49 -0300
Emile Cormier <emile.cormier.jr@gmail.com> wrote:
> From: ecorm <ecorm@users.noreply.github.com>
>
> Required by python-twisted[conch], which is required by python-crossbar.
> Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
> ---
> package/Config.in | 1 +
> package/python-appdirs/Config.in | 7 +++++++
> package/python-appdirs/python-appdirs.hash | 5 +++++
> package/python-appdirs/python-appdirs.mk | 14 ++++++++++++++
> 4 files changed, 27 insertions(+)
> create mode 100644 package/python-appdirs/Config.in
> create mode 100644 package/python-appdirs/python-appdirs.hash
> create mode 100644 package/python-appdirs/python-appdirs.mk
First of all, thanks a lot for your contribution, and sorry for the
slow feedback. I have now merged your patch, after:
- Fixing the e-mail in the From: field
- Adding the missing empty newline before the Signed-off-by line
- Adding the missing entry to the DEVELOPERS file
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* [Buildroot] [PATCH 3/6] package/python-zlmdb: add new package
2022-04-03 22:19 [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Emile Cormier
2022-04-03 22:19 ` [Buildroot] [PATCH 2/6] package/python-appdirs: add new package Emile Cormier
@ 2022-04-03 22:19 ` Emile Cormier
2022-08-07 14:53 ` Thomas Petazzoni via buildroot
2022-04-03 22:19 ` [Buildroot] [PATCH 4/6] package/python-twised: added conch and serial support Emile Cormier
` (3 subsequent siblings)
5 siblings, 1 reply; 21+ messages in thread
From: Emile Cormier @ 2022-04-03 22:19 UTC (permalink / raw)
To: buildroot
Cc: Asaf Kahlon, James Hilliard, Thomas Petazzoni, Emile Cormier,
Mauro Condarelli
From: ecorm <ecorm@users.noreply.github.com>
Required by python-crossbar.
Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
---
DEVELOPERS | 5 +++++
package/Config.in | 1 +
package/python-zlmdb/Config.in | 16 ++++++++++++++++
package/python-zlmdb/python-zlmdb.hash | 5 +++++
package/python-zlmdb/python-zlmdb.mk | 14 ++++++++++++++
5 files changed, 41 insertions(+)
create mode 100644 package/python-zlmdb/Config.in
create mode 100644 package/python-zlmdb/python-zlmdb.hash
create mode 100644 package/python-zlmdb/python-zlmdb.mk
diff --git a/DEVELOPERS b/DEVELOPERS
index a66b9d7eee..7f04663077 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -746,6 +746,11 @@ N: Eloi Bail <eloi.bail@savoirfairelinux.com>
F: package/bayer2rgb-neon/
F: package/gstreamer1/gst1-plugins-bayer2rgb-neon/
+N: Emile Cormier <emile.cormier.jr@gmail.com>
+F: package/python-autobahn/
+F: package/python-crossbar/
+F: package/python-zlmdb/
+
N: Eric Le Bihan <eric.le.bihan.dev@free.fr>
F: docs/manual/adding-packages-meson.txt
F: package/adwaita-icon-theme/
diff --git a/package/Config.in b/package/Config.in
index a1964df5f1..4330659431 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1300,6 +1300,7 @@ menu "External python modules"
source "package/python-yatl/Config.in"
source "package/python-zc-lockfile/Config.in"
source "package/python-zeroconf/Config.in"
+ source "package/python-zlmdb/Config.in"
source "package/python-zope-interface/Config.in"
source "package/python-zopfli/Config.in"
endmenu
diff --git a/package/python-zlmdb/Config.in b/package/python-zlmdb/Config.in
new file mode 100644
index 0000000000..9eaab3a7a7
--- /dev/null
+++ b/package/python-zlmdb/Config.in
@@ -0,0 +1,16 @@
+config BR2_PACKAGE_PYTHON_ZLMDB
+ bool "python-zlmdb"
+ select BR2_PACKAGE_PYTHON_CBOR2 # runtime
+ select BR2_PACKAGE_PYTHON_CFFI # runtime
+ select BR2_PACKAGE_PYTHON_CLICK # runtime
+ select BR2_PACKAGE_PYTHON_FLATBUFFERS # runtime
+ select BR2_PACKAGE_PYTHON_LMDB # runtime
+ select BR2_PACKAGE_PYTHON_NUMPY # runtime
+ select BR2_PACKAGE_PYTHON_PYNACL # runtime
+ select BR2_PACKAGE_PYTHON_PYYAML # runtime
+ select BR2_PACKAGE_PYTHON_TXAIO # runtime
+ help
+ Object-relational zero-copy in-memory database layer for
+ LMDB.
+
+ https://github.com/crossbario/zlmdb
diff --git a/package/python-zlmdb/python-zlmdb.hash b/package/python-zlmdb/python-zlmdb.hash
new file mode 100644
index 0000000000..d98dc7d12a
--- /dev/null
+++ b/package/python-zlmdb/python-zlmdb.hash
@@ -0,0 +1,5 @@
+# md5, sha256 from https://pypi.org/pypi/zlmdb/json
+md5 d730a394b74108933a65c9fbe76aba36 zlmdb-22.3.1.tar.gz
+sha256 0d416a47b1ee627186af8e5c99e5246f6b277df44ba5c6bb67e947eabf948e5a zlmdb-22.3.1.tar.gz
+# Locally computed sha256 checksums
+sha256 8a139bf325dcca11e0e476ce537bf791a6cfbeed8899a9b6ddc55238e180d230 LICENSE
diff --git a/package/python-zlmdb/python-zlmdb.mk b/package/python-zlmdb/python-zlmdb.mk
new file mode 100644
index 0000000000..a040622d9d
--- /dev/null
+++ b/package/python-zlmdb/python-zlmdb.mk
@@ -0,0 +1,14 @@
+################################################################################
+#
+# python-zlmdb
+#
+################################################################################
+
+PYTHON_ZLMDB_VERSION = 22.3.1
+PYTHON_ZLMDB_SOURCE = zlmdb-$(PYTHON_ZLMDB_VERSION).tar.gz
+PYTHON_ZLMDB_SITE = https://files.pythonhosted.org/packages/71/87/bb3cebd5312e670e33551317c7fc5e4b6a4a9af39075a71cd541b32cc0bf
+PYTHON_ZLMDB_SETUP_TYPE = setuptools
+PYTHON_ZLMDB_LICENSE = MIT
+PYTHON_ZLMDB_LICENSE_FILES = LICENSE
+
+$(eval $(python-package))
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [Buildroot] [PATCH 3/6] package/python-zlmdb: add new package
2022-04-03 22:19 ` [Buildroot] [PATCH 3/6] package/python-zlmdb: " Emile Cormier
@ 2022-08-07 14:53 ` Thomas Petazzoni via buildroot
2022-08-07 15:37 ` Emile Cormier
0 siblings, 1 reply; 21+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-08-07 14:53 UTC (permalink / raw)
To: Emile Cormier; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
Hello Emile,
I have also applied this patch, though with some changes. See below.
On Sun, 3 Apr 2022 19:19:50 -0300
Emile Cormier <emile.cormier.jr@gmail.com> wrote:
> From: ecorm <ecorm@users.noreply.github.com>
I fixed this bogus From: value.
>
> Required by python-crossbar.
Added an empty new line here.
> Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
> ---
> DEVELOPERS | 5 +++++
> package/Config.in | 1 +
> package/python-zlmdb/Config.in | 16 ++++++++++++++++
> package/python-zlmdb/python-zlmdb.hash | 5 +++++
> package/python-zlmdb/python-zlmdb.mk | 14 ++++++++++++++
> 5 files changed, 41 insertions(+)
> create mode 100644 package/python-zlmdb/Config.in
> create mode 100644 package/python-zlmdb/python-zlmdb.hash
> create mode 100644 package/python-zlmdb/python-zlmdb.mk
>
> diff --git a/DEVELOPERS b/DEVELOPERS
> index a66b9d7eee..7f04663077 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -746,6 +746,11 @@ N: Eloi Bail <eloi.bail@savoirfairelinux.com>
> F: package/bayer2rgb-neon/
> F: package/gstreamer1/gst1-plugins-bayer2rgb-neon/
>
> +N: Emile Cormier <emile.cormier.jr@gmail.com>
> +F: package/python-autobahn/
> +F: package/python-crossbar/
> +F: package/python-zlmdb/
Reduce this to just adding package/python-zlmdb/. Entries for existing
packages should be added in a separate commit.
> diff --git a/package/python-zlmdb/Config.in b/package/python-zlmdb/Config.in
> new file mode 100644
> index 0000000000..9eaab3a7a7
> --- /dev/null
> +++ b/package/python-zlmdb/Config.in
> @@ -0,0 +1,16 @@
> +config BR2_PACKAGE_PYTHON_ZLMDB
> + bool "python-zlmdb"
> + select BR2_PACKAGE_PYTHON_CBOR2 # runtime
> + select BR2_PACKAGE_PYTHON_CFFI # runtime
> + select BR2_PACKAGE_PYTHON_CLICK # runtime
> + select BR2_PACKAGE_PYTHON_FLATBUFFERS # runtime
> + select BR2_PACKAGE_PYTHON_LMDB # runtime
> + select BR2_PACKAGE_PYTHON_NUMPY # runtime
This one has some depends on that we need to replicate.
> + select BR2_PACKAGE_PYTHON_PYNACL # runtime
> + select BR2_PACKAGE_PYTHON_PYYAML # runtime
> + select BR2_PACKAGE_PYTHON_TXAIO # runtime
> + help
> + Object-relational zero-copy in-memory database layer for
> + LMDB.
> +
> + https://github.com/crossbario/zlmdb
So the final Config.in file looks like this:
config BR2_PACKAGE_PYTHON_ZLMDB
bool "python-zlmdb"
depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS # python-numpy
depends on BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL # python-numpy
select BR2_PACKAGE_PYTHON_CBOR2 # runtime
select BR2_PACKAGE_PYTHON_CFFI # runtime
select BR2_PACKAGE_PYTHON_CLICK # runtime
select BR2_PACKAGE_PYTHON_FLATBUFFERS # runtime
select BR2_PACKAGE_PYTHON_LMDB # runtime
select BR2_PACKAGE_PYTHON_NUMPY # runtime
select BR2_PACKAGE_PYTHON_PYNACL # runtime
select BR2_PACKAGE_PYTHON_PYYAML # runtime
select BR2_PACKAGE_PYTHON_TXAIO # runtime
help
Object-relational zero-copy in-memory database layer for
LMDB.
https://github.com/crossbario/zlmdb
comment "python-zlmdb needs glibc or musl"
depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
depends on !(BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL)
Another thing that this package really needs is a test case, in
support/testing/tests/package/. I was trying to look for some
documentation, but apart from a pretty obscure reference document that
doesn't have much explanations, I couldn't find anything.
Do you have some simple example on how to use zlmdb? This would allow
us to add a test case for this package that has a lot of runtime
dependencies.
Thanks!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread* Re: [Buildroot] [PATCH 3/6] package/python-zlmdb: add new package
2022-08-07 14:53 ` Thomas Petazzoni via buildroot
@ 2022-08-07 15:37 ` Emile Cormier
2022-08-07 16:10 ` Thomas Petazzoni via buildroot
0 siblings, 1 reply; 21+ messages in thread
From: Emile Cormier @ 2022-08-07 15:37 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
[-- Attachment #1.1: Type: text/plain, Size: 4683 bytes --]
Hi Thomas,
I'm in the process of migrating away from Crossbar, due to dynamic
authentication/authorization no longer working and the author not
interested in backporting bug fixes to older versions. I can no longer
devote any time to support Crossbar on Buildroot. Please do what you will
with these patch requests.
Sorry, I'm not familiar with zlmdb; the only reason I introduced it was to
satisfy a dependency by Crossbar.
Cheers,
Emile Cormier
On Sun, Aug 7, 2022 at 11:53 AM Thomas Petazzoni <
thomas.petazzoni@bootlin.com> wrote:
> Hello Emile,
>
> I have also applied this patch, though with some changes. See below.
>
> On Sun, 3 Apr 2022 19:19:50 -0300
> Emile Cormier <emile.cormier.jr@gmail.com> wrote:
>
> > From: ecorm <ecorm@users.noreply.github.com>
>
> I fixed this bogus From: value.
>
> >
> > Required by python-crossbar.
>
> Added an empty new line here.
>
> > Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
> > ---
> > DEVELOPERS | 5 +++++
> > package/Config.in | 1 +
> > package/python-zlmdb/Config.in | 16 ++++++++++++++++
> > package/python-zlmdb/python-zlmdb.hash | 5 +++++
> > package/python-zlmdb/python-zlmdb.mk | 14 ++++++++++++++
> > 5 files changed, 41 insertions(+)
> > create mode 100644 package/python-zlmdb/Config.in
> > create mode 100644 package/python-zlmdb/python-zlmdb.hash
> > create mode 100644 package/python-zlmdb/python-zlmdb.mk
> >
> > diff --git a/DEVELOPERS b/DEVELOPERS
> > index a66b9d7eee..7f04663077 100644
> > --- a/DEVELOPERS
> > +++ b/DEVELOPERS
> > @@ -746,6 +746,11 @@ N: Eloi Bail <eloi.bail@savoirfairelinux.com>
> > F: package/bayer2rgb-neon/
> > F: package/gstreamer1/gst1-plugins-bayer2rgb-neon/
> >
> > +N: Emile Cormier <emile.cormier.jr@gmail.com>
> > +F: package/python-autobahn/
> > +F: package/python-crossbar/
> > +F: package/python-zlmdb/
>
> Reduce this to just adding package/python-zlmdb/. Entries for existing
> packages should be added in a separate commit.
>
> > diff --git a/package/python-zlmdb/Config.in
> b/package/python-zlmdb/Config.in
> > new file mode 100644
> > index 0000000000..9eaab3a7a7
> > --- /dev/null
> > +++ b/package/python-zlmdb/Config.in
> > @@ -0,0 +1,16 @@
> > +config BR2_PACKAGE_PYTHON_ZLMDB
> > + bool "python-zlmdb"
> > + select BR2_PACKAGE_PYTHON_CBOR2 # runtime
> > + select BR2_PACKAGE_PYTHON_CFFI # runtime
> > + select BR2_PACKAGE_PYTHON_CLICK # runtime
> > + select BR2_PACKAGE_PYTHON_FLATBUFFERS # runtime
> > + select BR2_PACKAGE_PYTHON_LMDB # runtime
> > + select BR2_PACKAGE_PYTHON_NUMPY # runtime
>
> This one has some depends on that we need to replicate.
>
> > + select BR2_PACKAGE_PYTHON_PYNACL # runtime
> > + select BR2_PACKAGE_PYTHON_PYYAML # runtime
> > + select BR2_PACKAGE_PYTHON_TXAIO # runtime
> > + help
> > + Object-relational zero-copy in-memory database layer for
> > + LMDB.
> > +
> > + https://github.com/crossbario/zlmdb
>
> So the final Config.in file looks like this:
>
> config BR2_PACKAGE_PYTHON_ZLMDB
> bool "python-zlmdb"
> depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS # python-numpy
> depends on BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL #
> python-numpy
> select BR2_PACKAGE_PYTHON_CBOR2 # runtime
> select BR2_PACKAGE_PYTHON_CFFI # runtime
> select BR2_PACKAGE_PYTHON_CLICK # runtime
> select BR2_PACKAGE_PYTHON_FLATBUFFERS # runtime
> select BR2_PACKAGE_PYTHON_LMDB # runtime
> select BR2_PACKAGE_PYTHON_NUMPY # runtime
> select BR2_PACKAGE_PYTHON_PYNACL # runtime
> select BR2_PACKAGE_PYTHON_PYYAML # runtime
> select BR2_PACKAGE_PYTHON_TXAIO # runtime
> help
> Object-relational zero-copy in-memory database layer for
> LMDB.
>
> https://github.com/crossbario/zlmdb
>
> comment "python-zlmdb needs glibc or musl"
> depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
> depends on !(BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL)
>
> Another thing that this package really needs is a test case, in
> support/testing/tests/package/. I was trying to look for some
> documentation, but apart from a pretty obscure reference document that
> doesn't have much explanations, I couldn't find anything.
>
> Do you have some simple example on how to use zlmdb? This would allow
> us to add a test case for this package that has a lot of runtime
> dependencies.
>
> Thanks!
>
> Thomas
> --
> Thomas Petazzoni, co-owner and CEO, Bootlin
> Embedded Linux and Kernel engineering and training
> https://bootlin.com
>
[-- Attachment #1.2: Type: text/html, Size: 6527 bytes --]
[-- Attachment #2: Type: text/plain, Size: 150 bytes --]
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 3/6] package/python-zlmdb: add new package
2022-08-07 15:37 ` Emile Cormier
@ 2022-08-07 16:10 ` Thomas Petazzoni via buildroot
2022-08-07 16:50 ` Emile Cormier
0 siblings, 1 reply; 21+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-08-07 16:10 UTC (permalink / raw)
To: Emile Cormier; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
Hello,
On Sun, 7 Aug 2022 12:37:46 -0300
Emile Cormier <emile.cormier.jr@gmail.com> wrote:
> I'm in the process of migrating away from Crossbar, due to dynamic
> authentication/authorization no longer working and the author not
> interested in backporting bug fixes to older versions. I can no longer
> devote any time to support Crossbar on Buildroot. Please do what you will
> with these patch requests.
Ah, too bad :-/
Do you still have some time to comment on
https://patchwork.ozlabs.org/project/buildroot/patch/20220403221953.1158787-6-emile.cormier.jr@gmail.com/
? I was looking at it, and wanted to ask a few questions.
> Sorry, I'm not familiar with zlmdb; the only reason I introduced it was to
> satisfy a dependency by Crossbar.
ACK, no worries.
Best regards,
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 3/6] package/python-zlmdb: add new package
2022-08-07 16:10 ` Thomas Petazzoni via buildroot
@ 2022-08-07 16:50 ` Emile Cormier
0 siblings, 0 replies; 21+ messages in thread
From: Emile Cormier @ 2022-08-07 16:50 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
[-- Attachment #1.1: Type: text/plain, Size: 499 bytes --]
On Sun, Aug 7, 2022 at 1:10 PM Thomas Petazzoni <
thomas.petazzoni@bootlin.com> wrote:
> Do you still have some time to comment on
>
> https://patchwork.ozlabs.org/project/buildroot/patch/20220403221953.1158787-6-emile.cormier.jr@gmail.com/
> ? I was looking at it, and wanted to ask a few questions.
>
I'll try to answer your question as time allows. It's unfortunate this
wasn't done sooner while everything was still fresh in my mind, but I
understand that you might have had other priorities.
[-- Attachment #1.2: Type: text/html, Size: 980 bytes --]
[-- Attachment #2: Type: text/plain, Size: 150 bytes --]
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* [Buildroot] [PATCH 4/6] package/python-twised: added conch and serial support
2022-04-03 22:19 [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Emile Cormier
2022-04-03 22:19 ` [Buildroot] [PATCH 2/6] package/python-appdirs: add new package Emile Cormier
2022-04-03 22:19 ` [Buildroot] [PATCH 3/6] package/python-zlmdb: " Emile Cormier
@ 2022-04-03 22:19 ` Emile Cormier
2022-08-07 14:56 ` Thomas Petazzoni via buildroot
2022-04-03 22:19 ` [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages Emile Cormier
` (2 subsequent siblings)
5 siblings, 1 reply; 21+ messages in thread
From: Emile Cormier @ 2022-04-03 22:19 UTC (permalink / raw)
To: buildroot
Cc: Asaf Kahlon, James Hilliard, Thomas Petazzoni, Emile Cormier,
Mauro Condarelli
From: ecorm <ecorm@users.noreply.github.com>
The conch subpackage is required by python-crossbar.
Added the serial subpackage while I was at it (not required by
Crossbar).
Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
---
package/python-twisted/Config.in | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/package/python-twisted/Config.in b/package/python-twisted/Config.in
index 8596025e9c..c0acf35560 100644
--- a/package/python-twisted/Config.in
+++ b/package/python-twisted/Config.in
@@ -29,4 +29,16 @@ config BR2_PACKAGE_PYTHON_TWISTED_TLS
select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
select BR2_PACKAGE_PYTHON_SERVICE_IDENTITY # runtime
+config BR2_PACKAGE_PYTHON_TWISTED_CONCH
+ bool "Conch support"
+ depends on BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS # python-cryptography
+ select BR2_PACKAGE_PYTHON_APPDIRS # runtime
+ select BR2_PACKAGE_PYTHON_BCRYPT # runtime
+ select BR2_PACKAGE_PYTHON_CRYPTOGRAPHY # runtime
+ select BR2_PACKAGE_PYTHON_PYASN1 # runtime
+
+config BR2_PACKAGE_PYTHON_TWISTED_SERIAL
+ bool "Serial support"
+ select BR2_PACKAGE_PYTHON_SERIAL # runtime
+
endif
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 21+ messages in thread* [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages
2022-04-03 22:19 [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Emile Cormier
` (2 preceding siblings ...)
2022-04-03 22:19 ` [Buildroot] [PATCH 4/6] package/python-twised: added conch and serial support Emile Cormier
@ 2022-04-03 22:19 ` Emile Cormier
2022-08-07 16:12 ` Thomas Petazzoni via buildroot
2022-08-08 20:43 ` Thomas Petazzoni via buildroot
2022-04-03 22:19 ` [Buildroot] [PATCH 6/6] package/python-crossbar: fix configuration Emile Cormier
2022-04-04 19:47 ` [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Arnout Vandecappelle
5 siblings, 2 replies; 21+ messages in thread
From: Emile Cormier @ 2022-04-03 22:19 UTC (permalink / raw)
To: buildroot
Cc: Asaf Kahlon, James Hilliard, Thomas Petazzoni, Emile Cormier,
Mauro Condarelli
From: ecorm <ecorm@users.noreply.github.com>
Added missing python-autobahn subpackages required by python-crossbar.
Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
---
package/python-autobahn/Config.in | 40 +++++++++++++++++++++++++++++++
1 file changed, 40 insertions(+)
diff --git a/package/python-autobahn/Config.in b/package/python-autobahn/Config.in
index 627bcd73e5..20b1779b95 100644
--- a/package/python-autobahn/Config.in
+++ b/package/python-autobahn/Config.in
@@ -10,3 +10,43 @@ config BR2_PACKAGE_PYTHON_AUTOBAHN
framework.
https://pypi.python.org/pypi/autobahn
+
+if BR2_PACKAGE_PYTHON_AUTOBAHN
+
+config BR2_PACKAGE_PYTHON_AUTOBAHN_TWISTED
+ bool "Twisted support"
+ select BR2_PACKAGE_PYTHON_ATTRS # runtime
+ select BR2_PACKAGE_PYTHON_TWISTED # runtime
+ select BR2_PACKAGE_PYTHON_ZOPE_INTERFACE # runtime
+
+config BR2_PACKAGE_PYTHON_AUTOBAHN_ACCELERATE
+ bool "C-based WebSocket acceleration"
+ select BR2_PACKAGE_PYTHON_WSACCEL # runtime
+
+config BR2_PACKAGE_PYTHON_AUTOBAHN_COMPRESS
+ bool "Non-standard WebSocket compression support"
+ select BR2_PACKAGE_PYTHON_SNAPPY # runtime
+
+config BR2_PACKAGE_PYTHON_AUTOBAHN_SERIALIZATION
+ bool "Accelerated JSON, MessagePack, CBOR, UBJSON, and FlatBuffers serialization support"
+ select BR2_PACKAGE_PYTHON_CBOR # runtime
+ select BR2_PACKAGE_PYTHON_CBOR2 # runtime
+ select BR2_PACKAGE_PYTHON_FLATBUFFERS # runtime
+ select BR2_PACKAGE_PYTHON_MSGPACK # runtime
+ select BR2_PACKAGE_PYTHON_UBJSON # runtime
+ select BR2_PACKAGE_PYTHON_UJSON # runtime
+
+config BR2_PACKAGE_PYTHON_AUTOBAHN_ENCRYPTION
+ bool "TLS and WAMP-cryptosign encryption/authentication support"
+ select BR2_PACKAGE_PYTHON_PYNACL # runtime
+ select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
+ select BR2_PACKAGE_PYTHON_PYQRCODE # runtime
+ select BR2_PACKAGE_PYTHON_PYTRIE # runtime
+ select BR2_PACKAGE_PYTHON_SERVICE_IDENTITY # runtime
+
+config BR2_PACKAGE_PYTHON_AUTOBAHN_SCRAM
+ bool "WAMP-SCRAM authentication support"
+ select BR2_PACKAGE_PYTHON_ARGON2_CFFI # runtime
+ select BR2_PACKAGE_PYTHON_PASSLIB # runtime
+
+endif
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages
2022-04-03 22:19 ` [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages Emile Cormier
@ 2022-08-07 16:12 ` Thomas Petazzoni via buildroot
2022-08-07 16:46 ` Emile Cormier
2022-08-08 20:43 ` Thomas Petazzoni via buildroot
1 sibling, 1 reply; 21+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-08-07 16:12 UTC (permalink / raw)
To: Emile Cormier; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
Hello Emile,
I have applied, with a few changes, see below.
On Sun, 3 Apr 2022 19:19:52 -0300
Emile Cormier <emile.cormier.jr@gmail.com> wrote:
> From: ecorm <ecorm@users.noreply.github.com>
>
> Added missing python-autobahn subpackages required by python-crossbar.
> Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
Fixed up the From: and missing empty line, as usual.
> +if BR2_PACKAGE_PYTHON_AUTOBAHN
> +
> +config BR2_PACKAGE_PYTHON_AUTOBAHN_TWISTED
> + bool "Twisted support"
> + select BR2_PACKAGE_PYTHON_ATTRS # runtime
> + select BR2_PACKAGE_PYTHON_TWISTED # runtime
> + select BR2_PACKAGE_PYTHON_ZOPE_INTERFACE # runtime
> +
> +config BR2_PACKAGE_PYTHON_AUTOBAHN_ACCELERATE
> + bool "C-based WebSocket acceleration"
> + select BR2_PACKAGE_PYTHON_WSACCEL # runtime
> +
> +config BR2_PACKAGE_PYTHON_AUTOBAHN_COMPRESS
> + bool "Non-standard WebSocket compression support"
> + select BR2_PACKAGE_PYTHON_SNAPPY # runtime
> +
> +config BR2_PACKAGE_PYTHON_AUTOBAHN_SERIALIZATION
> + bool "Accelerated JSON, MessagePack, CBOR, UBJSON, and FlatBuffers serialization support"
> + select BR2_PACKAGE_PYTHON_CBOR # runtime
I do see the cbor2 dependency being expressed in setup.py, but not the
cbor dependency. Also, I only cbor2 being imported in the autobahn code
base, so I dropped this line. Of course, if there is something I
missed, let me know.
> + select BR2_PACKAGE_PYTHON_CBOR2 # runtime
> + select BR2_PACKAGE_PYTHON_FLATBUFFERS # runtime
> + select BR2_PACKAGE_PYTHON_MSGPACK # runtime
> + select BR2_PACKAGE_PYTHON_UBJSON # runtime
> + select BR2_PACKAGE_PYTHON_UJSON # runtime
> +
> +config BR2_PACKAGE_PYTHON_AUTOBAHN_ENCRYPTION
> + bool "TLS and WAMP-cryptosign encryption/authentication support"
> + select BR2_PACKAGE_PYTHON_PYNACL # runtime
> + select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
> + select BR2_PACKAGE_PYTHON_PYQRCODE # runtime
> + select BR2_PACKAGE_PYTHON_PYTRIE # runtime
> + select BR2_PACKAGE_PYTHON_SERVICE_IDENTITY # runtime
> +
> +config BR2_PACKAGE_PYTHON_AUTOBAHN_SCRAM
> + bool "WAMP-SCRAM authentication support"
> + select BR2_PACKAGE_PYTHON_ARGON2_CFFI # runtime
> + select BR2_PACKAGE_PYTHON_PASSLIB # runtime
The other thing I changed is to sort those sub-options alphabetically
by their option name.
Applied with those changes!
Thanks,
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages
2022-08-07 16:12 ` Thomas Petazzoni via buildroot
@ 2022-08-07 16:46 ` Emile Cormier
2022-08-07 17:18 ` Thomas Petazzoni via buildroot
0 siblings, 1 reply; 21+ messages in thread
From: Emile Cormier @ 2022-08-07 16:46 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
[-- Attachment #1.1: Type: text/plain, Size: 821 bytes --]
On Sun, Aug 7, 2022 at 1:12 PM Thomas Petazzoni <
thomas.petazzoni@bootlin.com> wrote:
> I do see the cbor2 dependency being expressed in setup.py, but not the
> cbor dependency. Also, I only cbor2 being imported in the autobahn code
> base, so I dropped this line. Of course, if there is something I
> missed, let me know.
>
Are you sure you were checking in the 22.1.1 version of Autobahn?
The cbor dependency is here:
https://github.com/crossbario/autobahn-python/blob/dc626948d5ffbd741ec1e3a7906870eae1a4c69b/setup.py#L82
"import cbor" is used in both these places:
https://github.com/crossbario/autobahn-python/blob/dc626948d5ffbd741ec1e3a7906870eae1a4c69b/autobahn/wamp/message.py#L36
https://github.com/crossbario/autobahn-python/blob/dc626948d5ffbd741ec1e3a7906870eae1a4c69b/autobahn/wamp/serializer.py#L660
[-- Attachment #1.2: Type: text/html, Size: 1638 bytes --]
[-- Attachment #2: Type: text/plain, Size: 150 bytes --]
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages
2022-08-07 16:46 ` Emile Cormier
@ 2022-08-07 17:18 ` Thomas Petazzoni via buildroot
2022-08-07 17:37 ` Emile Cormier
0 siblings, 1 reply; 21+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-08-07 17:18 UTC (permalink / raw)
To: Emile Cormier; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
Hello Emile,
On Sun, 7 Aug 2022 13:46:21 -0300
Emile Cormier <emile.cormier.jr@gmail.com> wrote:
> Are you sure you were checking in the 22.1.1 version of Autobahn?
No, I'm checking in version 22.6.1, because that's the one we have in
Buildroot now.
> The cbor dependency is here:
> https://github.com/crossbario/autobahn-python/blob/dc626948d5ffbd741ec1e3a7906870eae1a4c69b/setup.py#L82
>
> "import cbor" is used in both these places:
> https://github.com/crossbario/autobahn-python/blob/dc626948d5ffbd741ec1e3a7906870eae1a4c69b/autobahn/wamp/message.py#L36
> https://github.com/crossbario/autobahn-python/blob/dc626948d5ffbd741ec1e3a7906870eae1a4c69b/autobahn/wamp/serializer.py#L660
Looking at 22.6.1, I think cbor is really no longer used:
$ git grep "import.*cbor.*"
autobahn/wamp/message.py: import cbor2
autobahn/wamp/serializer.py: import cbor2
autobahn/xbr/_buyer.py:import cbor2
autobahn/xbr/_cli.py:import cbor2
autobahn/xbr/_seller.py:import cbor2
autobahn/xbr/templates/py-autobahn/test_obj.py.jinja2:import cbor2
Am I missing something?
Thanks!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages
2022-08-07 17:18 ` Thomas Petazzoni via buildroot
@ 2022-08-07 17:37 ` Emile Cormier
2022-08-09 8:15 ` Thomas Petazzoni via buildroot
0 siblings, 1 reply; 21+ messages in thread
From: Emile Cormier @ 2022-08-07 17:37 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
[-- Attachment #1.1: Type: text/plain, Size: 840 bytes --]
On Sun, Aug 7, 2022 at 2:18 PM Thomas Petazzoni <
thomas.petazzoni@bootlin.com> wrote:
> > Are you sure you were checking in the 22.1.1 version of Autobahn?
>
> No, I'm checking in version 22.6.1, because that's the one we have in
> Buildroot now.
>
Oh I see. My patch request was for the Autobahn v22.1.1 that was included
in Buildroot at the time. If the Autobahn version was bumped up to 22.6.1
then you'll have to recheck all the dependencies if you haven't already.
If you're planning on bumping Crossbar to the latest, be warned that you'll
end up in dependency hell as it introduces a bunch of new dependencies for
the XBR and crypto features. Crossbar performs checks at runtime to bracket
the versions of these dependencies and you may have to patch some of the
dependency version ranges (or patch the runtime version checks).
[-- Attachment #1.2: Type: text/html, Size: 1216 bytes --]
[-- Attachment #2: Type: text/plain, Size: 150 bytes --]
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages
2022-08-07 17:37 ` Emile Cormier
@ 2022-08-09 8:15 ` Thomas Petazzoni via buildroot
0 siblings, 0 replies; 21+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-08-09 8:15 UTC (permalink / raw)
To: Emile Cormier; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
Hello Emile,
On Sun, 7 Aug 2022 14:37:59 -0300
Emile Cormier <emile.cormier.jr@gmail.com> wrote:
> Oh I see. My patch request was for the Autobahn v22.1.1 that was included
> in Buildroot at the time. If the Autobahn version was bumped up to 22.6.1
> then you'll have to recheck all the dependencies if you haven't already.
>
> If you're planning on bumping Crossbar to the latest, be warned that you'll
> end up in dependency hell as it introduces a bunch of new dependencies for
> the XBR and crypto features. Crossbar performs checks at runtime to bracket
> the versions of these dependencies and you may have to patch some of the
> dependency version ranges (or patch the runtime version checks).
At this point, I do not intend to bump Crossbar. I'll leave that to
people who are interested in this :-)
Best regards,
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages
2022-04-03 22:19 ` [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages Emile Cormier
2022-08-07 16:12 ` Thomas Petazzoni via buildroot
@ 2022-08-08 20:43 ` Thomas Petazzoni via buildroot
1 sibling, 0 replies; 21+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-08-08 20:43 UTC (permalink / raw)
To: Emile Cormier; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
Hello Emile,
On Sun, 3 Apr 2022 19:19:52 -0300
Emile Cormier <emile.cormier.jr@gmail.com> wrote:
> +config BR2_PACKAGE_PYTHON_AUTOBAHN_ENCRYPTION
> + bool "TLS and WAMP-cryptosign encryption/authentication support"
> + select BR2_PACKAGE_PYTHON_PYNACL # runtime
> + select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
> + select BR2_PACKAGE_PYTHON_PYQRCODE # runtime
Turned out this line was not correct! Crossbar indeed uses pyqrcode,
but autobahn uses qrcode, which is a different Python module.
So I packaged qrcode:
https://git.buildroot.org/buildroot/commit/?id=37f05c2497a05c9d007ed03c224f4a4d18a21b21
And then fixed autobahn to use it:
https://git.buildroot.org/buildroot/commit/?id=6826595759d1978633008479d305abb8a894d442
Thanks!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* [Buildroot] [PATCH 6/6] package/python-crossbar: fix configuration
2022-04-03 22:19 [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Emile Cormier
` (3 preceding siblings ...)
2022-04-03 22:19 ` [Buildroot] [PATCH 5/6] package/python-autobahn: add missing subpackages Emile Cormier
@ 2022-04-03 22:19 ` Emile Cormier
2022-08-08 20:45 ` Thomas Petazzoni via buildroot
2022-04-04 19:47 ` [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Arnout Vandecappelle
5 siblings, 1 reply; 21+ messages in thread
From: Emile Cormier @ 2022-04-03 22:19 UTC (permalink / raw)
To: buildroot
Cc: Asaf Kahlon, James Hilliard, Thomas Petazzoni, Emile Cormier,
Mauro Condarelli
From: ecorm <ecorm@users.noreply.github.com>
Fixes configuration and patches setup requirements so that
python-crossbar can build and run.
See https://github.com/crossbario/crossbar/issues/1936 for discussion
with Crossbar author.
Fixes: #14556
Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
---
.../0002-Remove-idna-requirement.patch | 29 ----------
...002-Resolve-conflicting-requirements.patch | 57 +++++++++++++++++++
package/python-crossbar/Config.in | 21 +++----
3 files changed, 68 insertions(+), 39 deletions(-)
delete mode 100644 package/python-crossbar/0002-Remove-idna-requirement.patch
create mode 100644 package/python-crossbar/0002-Resolve-conflicting-requirements.patch
diff --git a/package/python-crossbar/0002-Remove-idna-requirement.patch b/package/python-crossbar/0002-Remove-idna-requirement.patch
deleted file mode 100644
index 5e19a467c2..0000000000
--- a/package/python-crossbar/0002-Remove-idna-requirement.patch
+++ /dev/null
@@ -1,29 +0,0 @@
-From 9164d21cc66c1f78de37c9383528c5d528cbdbeb Mon Sep 17 00:00:00 2001
-From: Asaf Kahlon <asafka7@gmail.com>
-Date: Wed, 26 Sep 2018 15:33:43 +0300
-Subject: [PATCH] Remove idna requirement.
-
-The latest version of idna is 2.7, but it seems like idna is not a direct
-dependency anymore (a short "git grep" shows it's only written in requirement
-and readme files).
-
-Signed-off-by: Asaf Kahlon <asafka7@gmail.com>
----
- requirements-min.txt | 1 -
- 1 file changed, 1 deletion(-)
-
-diff --git a/requirements-min.txt b/requirements-min.txt
-index 92fb0f8f..13cdac87 100644
---- a/requirements-min.txt
-+++ b/requirements-min.txt
-@@ -8,7 +8,6 @@ click>=6.7
- constantly>=15.1.0
- cryptography>=2.6.1
- h2>=3.2.0
--idna<2.6,>=2.5
- importlib-resources>=4.1.1
- incremental>=17.5.0
- jinja2>=2.10.1
---
-2.17.1
-
diff --git a/package/python-crossbar/0002-Resolve-conflicting-requirements.patch b/package/python-crossbar/0002-Resolve-conflicting-requirements.patch
new file mode 100644
index 0000000000..afb182cab1
--- /dev/null
+++ b/package/python-crossbar/0002-Resolve-conflicting-requirements.patch
@@ -0,0 +1,57 @@
+Remove the upper limit for idna, which is an indirect dependency via
+twisted[tls] and treq->requests. This upper limit conflicts with the
+python-idna v3.3 package bundled with buildroot. The idna requirement for
+twisted v20.3.0 is idna>=0.6,!=2.3 , and for requests it's
+idna>=2.5,<4;python_version>="3" .
+
+Remove the upper limit for urllib3, which is an indirect dependency via
+treq->requests. This upper limit conflicts with the python-urllib3 v1.26.8
+package bundled with buildroot. The urllib3 requirements for requests v2.27.1
+is urllib3>=1.21.1,<1.27 .
+
+Set the lower limit for urllib3 to >1.24.2 to satisfy the comment regarding
+CVE-2019-11324.
+
+Remove the importlib-resources>=4.1.1 requirement. The features of
+importlib-resources v5.0 (and thus v4.1.1) have been merged into the
+CPython 3.10 bundled with buildroot.
+
+Remove the u-msgpack-python requirement, which is an indirect dependency via
+autobahn[serialization] that is not needed due to the python-msgpack package
+being chosen instead on CPython.
+
+Remove the netaddr requirement, which is an indirect dependency via
+autobahn[xbr]->web3. buildroot's python-autobahn package currently does not
+support the autobahn[xbr] extras.
+
+Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
+
+--- a/requirements-min.txt
++++ b/requirements-min.txt
+@@ -8,13 +8,11 @@ colorama>=0.4.4
+ constantly>=15.1.0
+ cryptography>=2.6.1
+ h2>=3.2.0
+-idna<2.6,>=2.5
+-importlib-resources>=4.1.1
++idna>=2.5
+ incremental>=17.5.0
+ jinja2>=2.10.1
+ lmdb>=0.92
+ mistune>=0.7.4
+-netaddr>=0.7.19
+ passlib>=1.7.1
+ priority>=1.3.0
+ psutil>=5.2.2
+@@ -37,10 +35,9 @@ twisted[tls,conch,http2,osx_platform]>=2
+ twisted[tls,conch,http2,windows_platform]>=20.3.0; sys_platform == 'win32'
+ txaio>=21.2.1
+ txtorcon>=20.0.0
+-u-msgpack-python>=2.4.1
+ # urllib3 is an indirect dependency, but we force a recent version because of https://nvd.nist.gov/vuln/detail/CVE-2019-11324
+ # workaround for version conflict in requests vs sth else:
+-urllib3<1.25,>=1.21.1
++urllib3>1.24.2
+ vmprof>=0.4.12; platform_machine=='x86_64' or platform_machine=='i386' or platform_machine=='arm'
+ watchdog>=0.8.3
+ werkzeug>=0.14.1
diff --git a/package/python-crossbar/Config.in b/package/python-crossbar/Config.in
index 9625d9ce8e..ef02dd54f9 100644
--- a/package/python-crossbar/Config.in
+++ b/package/python-crossbar/Config.in
@@ -2,21 +2,21 @@ config BR2_PACKAGE_PYTHON_CROSSBAR
bool "python-crossbar"
depends on BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS # python-cryptography
# All the following dependencies are runtime dependencies
- select BR2_PACKAGE_PYTHON_ATTRS
select BR2_PACKAGE_PYTHON_AUTOBAHN
+ select BR2_PACKAGE_PYTHON_AUTOBAHN_ACCELERATE
+ select BR2_PACKAGE_PYTHON_AUTOBAHN_COMPRESS
+ select BR2_PACKAGE_PYTHON_AUTOBAHN_ENCRYPTION
+ select BR2_PACKAGE_PYTHON_AUTOBAHN_SCRAM
+ select BR2_PACKAGE_PYTHON_AUTOBAHN_SERIALIZATION
+ select BR2_PACKAGE_PYTHON_AUTOBAHN_TWISTED
select BR2_PACKAGE_PYTHON_BITSTRING
select BR2_PACKAGE_PYTHON_CBOR
- select BR2_PACKAGE_PYTHON_CBOR2
select BR2_PACKAGE_PYTHON_CLICK
- select BR2_PACKAGE_PYTHON_CONSTANTLY
+ select BR2_PACKAGE_PYTHON_COLORAMA
select BR2_PACKAGE_PYTHON_CRYPTOGRAPHY
- select BR2_PACKAGE_PYTHON_H2
- select BR2_PACKAGE_PYTHON_IDNA
- select BR2_PACKAGE_PYTHON_INCREMENTAL
select BR2_PACKAGE_PYTHON_JINJA2
select BR2_PACKAGE_PYTHON_LMDB
select BR2_PACKAGE_PYTHON_MISTUNE
- select BR2_PACKAGE_PYTHON_NETADDR
select BR2_PACKAGE_PYTHON_PASSLIB
select BR2_PACKAGE_PYTHON_PRIORITY
select BR2_PACKAGE_PYTHON_PSUTIL
@@ -29,17 +29,18 @@ config BR2_PACKAGE_PYTHON_CROSSBAR
select BR2_PACKAGE_PYTHON_PYTRIE
select BR2_PACKAGE_PYTHON_PYYAML
select BR2_PACKAGE_PYTHON_SDNOTIFY
- select BR2_PACKAGE_PYTHON_SERVICE_IDENTITY
select BR2_PACKAGE_PYTHON_SETPROCTITLE
select BR2_PACKAGE_PYTHON_SETUPTOOLS
select BR2_PACKAGE_PYTHON_TREQ
select BR2_PACKAGE_PYTHON_TWISTED
+ select BR2_PACKAGE_PYTHON_TWISTED_CONCH
+ select BR2_PACKAGE_PYTHON_TWISTED_HTTP2
+ select BR2_PACKAGE_PYTHON_TWISTED_TLS
select BR2_PACKAGE_PYTHON_TXAIO
select BR2_PACKAGE_PYTHON_TXTORCON
- select BR2_PACKAGE_PYTHON_U_MSGPACK
- select BR2_PACKAGE_PYTHON_UBJSON
select BR2_PACKAGE_PYTHON_WATCHDOG
select BR2_PACKAGE_PYTHON_WERKZEUG
+ select BR2_PACKAGE_PYTHON_ZLMDB
select BR2_PACKAGE_PYTHON_ZOPE_INTERFACE
help
Crossbar.io is an open-source WAMP application router that
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [Buildroot] [PATCH 6/6] package/python-crossbar: fix configuration
2022-04-03 22:19 ` [Buildroot] [PATCH 6/6] package/python-crossbar: fix configuration Emile Cormier
@ 2022-08-08 20:45 ` Thomas Petazzoni via buildroot
2022-08-08 21:39 ` Emile Cormier
0 siblings, 1 reply; 21+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-08-08 20:45 UTC (permalink / raw)
To: Emile Cormier; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
Hello Emile,
On Sun, 3 Apr 2022 19:19:53 -0300
Emile Cormier <emile.cormier.jr@gmail.com> wrote:
> From: ecorm <ecorm@users.noreply.github.com>
>
> Fixes configuration and patches setup requirements so that
> python-crossbar can build and run.
> See https://github.com/crossbario/crossbar/issues/1936 for discussion
> with Crossbar author.
> Fixes: #14556
> Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
> ---
> .../0002-Remove-idna-requirement.patch | 29 ----------
> ...002-Resolve-conflicting-requirements.patch | 57 +++++++++++++++++++
> package/python-crossbar/Config.in | 21 +++----
> 3 files changed, 68 insertions(+), 39 deletions(-)
> delete mode 100644 package/python-crossbar/0002-Remove-idna-requirement.patch
> create mode 100644 package/python-crossbar/0002-Resolve-conflicting-requirements.patch
I have applied, with a number of changes:
- The patch 0002-Resolve-conflicting-requirements.patch is replaced by
a more general patch that removes indirect dependencies from
requirements-min.txt.
- The test case for crossbar is changed to no longer force autobahn to
use umsgpack, as we now use msgpack, expected by default by autobahn.
The minimal Crossbar test case we have in
https://git.buildroot.org/buildroot/tree/support/testing/tests/package/sample_python_crossbar.py
runs correctly.
If you have some suggestions of other things to test (simple tests) to
better validate the Crossbar installation, it would be useful.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 6/6] package/python-crossbar: fix configuration
2022-08-08 20:45 ` Thomas Petazzoni via buildroot
@ 2022-08-08 21:39 ` Emile Cormier
0 siblings, 0 replies; 21+ messages in thread
From: Emile Cormier @ 2022-08-08 21:39 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: James Hilliard, Mauro Condarelli, Asaf Kahlon, buildroot
[-- Attachment #1.1: Type: text/plain, Size: 2360 bytes --]
Hi Thomas,
Thank you for your efforts towards fixing Buildroot's Crossbar. Although I
am migrating away from Crossbar (will be writing my own embeddable C++
router), I'm sure it may be useful to others.
"crossbar status" is pretty much the only easy test I know off the top of
my head. There might be some stuff in Crossbar's test suite that could be
borrowed, but I don't have much time to check right now:
https://github.com/crossbario/crossbar/tree/master/test
My expertise is more in C++ and I am not fluent in Python (although I can
read the latter).
Cheers,
Emile Cormier
On Mon, Aug 8, 2022 at 5:45 PM Thomas Petazzoni <
thomas.petazzoni@bootlin.com> wrote:
> Hello Emile,
>
> On Sun, 3 Apr 2022 19:19:53 -0300
> Emile Cormier <emile.cormier.jr@gmail.com> wrote:
>
> > From: ecorm <ecorm@users.noreply.github.com>
> >
> > Fixes configuration and patches setup requirements so that
> > python-crossbar can build and run.
> > See https://github.com/crossbario/crossbar/issues/1936 for discussion
> > with Crossbar author.
> > Fixes: #14556
> > Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
> > ---
> > .../0002-Remove-idna-requirement.patch | 29 ----------
> > ...002-Resolve-conflicting-requirements.patch | 57 +++++++++++++++++++
> > package/python-crossbar/Config.in | 21 +++----
> > 3 files changed, 68 insertions(+), 39 deletions(-)
> > delete mode 100644
> package/python-crossbar/0002-Remove-idna-requirement.patch
> > create mode 100644
> package/python-crossbar/0002-Resolve-conflicting-requirements.patch
>
> I have applied, with a number of changes:
>
> - The patch 0002-Resolve-conflicting-requirements.patch is replaced by
> a more general patch that removes indirect dependencies from
> requirements-min.txt.
>
> - The test case for crossbar is changed to no longer force autobahn to
> use umsgpack, as we now use msgpack, expected by default by autobahn.
>
> The minimal Crossbar test case we have in
>
> https://git.buildroot.org/buildroot/tree/support/testing/tests/package/sample_python_crossbar.py
> runs correctly.
>
> If you have some suggestions of other things to test (simple tests) to
> better validate the Crossbar installation, it would be useful.
>
> Thanks!
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
[-- Attachment #1.2: Type: text/html, Size: 3597 bytes --]
[-- Attachment #2: Type: text/plain, Size: 150 bytes --]
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1
2022-04-03 22:19 [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Emile Cormier
` (4 preceding siblings ...)
2022-04-03 22:19 ` [Buildroot] [PATCH 6/6] package/python-crossbar: fix configuration Emile Cormier
@ 2022-04-04 19:47 ` Arnout Vandecappelle
5 siblings, 0 replies; 21+ messages in thread
From: Arnout Vandecappelle @ 2022-04-04 19:47 UTC (permalink / raw)
To: Emile Cormier, buildroot
Cc: James Hilliard, Thomas Petazzoni, Asaf Kahlon, Mauro Condarelli
On 04/04/2022 00:19, Emile Cormier wrote:
> From: ecorm <ecorm@users.noreply.github.com>
Meh, annoying that github does this.
>
> Required by python-crossbar and python-autobahn[compress].
> Signed-off-by: Emile Cormier <emile.cormier.jr@gmail.com>
Please make sure there's an empty line before your Signed-off-by.
> ---
> package/python-snappy/python-snappy.hash | 4 ++--
> package/python-snappy/python-snappy.mk | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/package/python-snappy/python-snappy.hash b/package/python-snappy/python-snappy.hash
> index 1e0f96054f..32df2bea48 100644
> --- a/package/python-snappy/python-snappy.hash
> +++ b/package/python-snappy/python-snappy.hash
> @@ -1,5 +1,5 @@
> # md5, sha256 from https://pypi.org/pypi/python-snappy/json
> -md5 82e09ffd44204096fc70d6de14d0ea2f python-snappy-0.5.4.tar.gz
> -sha256 d9c26532cfa510f45e8d135cde140e8a5603d3fb254cfec273ebc0ecf9f668e2 python-snappy-0.5.4.tar.gz
> +md5 fc84cfa3526fba06d66f0d0b2e764300 python-snappy-0.6.1.tar.gz
> +sha256 b6a107ab06206acc5359d4c5632bd9b22d448702a79b3169b0c62e0fb808bb2a python-snappy-0.6.1.tar.gz
When updating a hash file, please also update the spacing to two spaces (as
reported by check-package).
Applied to master with all of the above fixed, thanks.
Regards,
Arnout
> # Locally computed sha256 checksums
> sha256 d007cb05f143d99d16d5cd0d1bf291daf037adb36800344e440ec26edb938f9a LICENSE
> diff --git a/package/python-snappy/python-snappy.mk b/package/python-snappy/python-snappy.mk
> index d087c83b10..e80aa861df 100644
> --- a/package/python-snappy/python-snappy.mk
> +++ b/package/python-snappy/python-snappy.mk
> @@ -4,8 +4,8 @@
> #
> ################################################################################
>
> -PYTHON_SNAPPY_VERSION = 0.5.4
> -PYTHON_SNAPPY_SITE = https://files.pythonhosted.org/packages/45/35/65d9f8cc537129894b4b32647d80212d1fa342877581c5b8a69872cea8be
> +PYTHON_SNAPPY_VERSION = 0.6.1
> +PYTHON_SNAPPY_SITE = https://files.pythonhosted.org/packages/98/7a/44a24bad98335b2c72e4cadcdecf79f50197d1bab9f22f863a274f104b96
> PYTHON_SNAPPY_SETUP_TYPE = setuptools
> PYTHON_SNAPPY_LICENSE = BSD-3-Clause
> PYTHON_SNAPPY_LICENSE_FILES = LICENSE
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 21+ messages in thread