All of 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 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.