[Buildroot] [PATCH 1/3] python3-m2crypto: import package

Thomas Petazzoni thomas.petazzoni at bootlin.com
Thu Jun 18 20:24:32 UTC 2020


Hello Kurt,

Thanks for this contribution. The commit title needs to be:

	package/python3-m2crypto: new package

It should be mentioned that we used to have a python-m2crypto package,
which was removed in commit 484c04878e5e76cdc0cc33c7646f98ab41698a7e
because it was only used by crda, and crda no longer needed it.

On Fri, 12 Jun 2020 07:31:08 +0200
Kurt Van Dijck <dev.kurt at vandijck-laurijssen.be> wrote:

> Signed-off-by: Kurt Van Dijck <dev.kurt at vandijck-laurijssen.be>
> ---
>  package/python3-m2crypto/python3-m2crypto.mk | 21 ++++++++++++++++++++
>  1 file changed, 21 insertions(+)

You're missing a .hash file, as well as an entry in the DEVELOPERS file.

>  create mode 100644 package/python3-m2crypto/python3-m2crypto.mk
> 
> diff --git a/package/python3-m2crypto/python3-m2crypto.mk b/package/python3-m2crypto/python3-m2crypto.mk
> new file mode 100644
> index 0000000000..4394226067
> --- /dev/null
> +++ b/package/python3-m2crypto/python3-m2crypto.mk
> @@ -0,0 +1,21 @@
> +################################################################################
> +#
> +# python3-m2crypto
> +#
> +################################################################################
> +
> +# Please keep in sync with

in sync with ?

> +PYTHON3_M2CRYPTO_VERSION = 0.35.2
> +PYTHON3_M2CRYPTO_SOURCE = M2Crypto-$(PYTHON3_M2CRYPTO_VERSION).tar.gz
> +PYTHON3_M2CRYPTO_SITE = https://files.pythonhosted.org/packages/74/18/3beedd4ac48b52d1a4d12f2a8c5cf0ae342ce974859fba838cbbc1580249
> +PYTHON3_M2CRYPTO_SETUP_TYPE = setuptools
> +PYTHON3_M2CRYPTO_LICENSE = MIT
> +PYTHON3_M2CRYPTO_LICENSE_FILES = LICENSE
> +HOST_PYTHON3_M2CRYPTO_DL_SUBDIR = python-m2crypto
> +HOST_PYTHON3_M2CRYPTO_NEEDS_HOST_PYTHON = python3
> +
> +HOST_PYTHON3_M2CRYPTO_DEPENDENCIES = host-swig
> +
> +HOST_PYTHON3_M2CRYPTO_BUILD_OPTS = --openssl $(HOST_DIR)

It needs OpenSSL ? Then I guess host-openssl should be in the dependencies, no ?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


More information about the buildroot mailing list