Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Emile Cormier <emile.cormier.jr@gmail.com>
Cc: James Hilliard <james.hilliard1@gmail.com>,
	Mauro Condarelli <mc5686@mclink.it>,
	Asaf Kahlon <asafka7@gmail.com>,
	buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 3/6] package/python-zlmdb: add new package
Date: Sun, 7 Aug 2022 16:53:00 +0200	[thread overview]
Message-ID: <20220807165300.2137b391@windsurf> (raw)
In-Reply-To: <20220403221953.1158787-3-emile.cormier.jr@gmail.com>

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

  reply	other threads:[~2022-08-07 14:53 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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-08-07 14:50   ` Thomas Petazzoni via buildroot
2022-04-03 22:19 ` [Buildroot] [PATCH 3/6] package/python-zlmdb: " Emile Cormier
2022-08-07 14:53   ` Thomas Petazzoni via buildroot [this message]
2022-08-07 15:37     ` Emile Cormier
2022-08-07 16:10       ` Thomas Petazzoni via buildroot
2022-08-07 16:50         ` Emile Cormier
2022-04-03 22:19 ` [Buildroot] [PATCH 4/6] package/python-twised: added conch and serial support 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
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
2022-08-07 17:37         ` Emile Cormier
2022-08-09  8:15           ` 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-08-08 20:45   ` Thomas Petazzoni via buildroot
2022-08-08 21:39     ` Emile Cormier
2022-04-04 19:47 ` [Buildroot] [PATCH 1/6] package/python-snappy: bump version to 0.6.1 Arnout Vandecappelle

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220807165300.2137b391@windsurf \
    --to=buildroot@buildroot.org \
    --cc=asafka7@gmail.com \
    --cc=emile.cormier.jr@gmail.com \
    --cc=james.hilliard1@gmail.com \
    --cc=mc5686@mclink.it \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox