Re: [NEW] archivers/py-lz4

On Wed 02/09/2020 13:09, Remi Pointel wrote:
> Hi,
>
> this is the port to have python bindings for lz4.
>
> This is needed by the plaso's update.
>
> Ok?
>
> Cheers,
>
> Remi.

Initial port does not build for me.

With the diff below I'm able to build and test. Instead of using the
bundled version of lz4 I propose to use the one that we have in ports.


diff --git Makefile Makefile
index 21fd619dd7f..995172fa6f6 100644
--- Makefile
+++ Makefile
@@ -12,15 +12,22 @@ CATEGORIES = archivers
# BSD
PERMIT_PACKAGE = Yes

+WANTLIB = lz4 pthread
+
MODULES = lang/python

MODPY_PI = Yes
MODPY_SETUPTOOLS = Yes
+MODPY_PYTEST = Yes
+
+BUILD_DEPENDS = devel/py-pkgconfig${MODPY_FLAVOR} \
+ devel/py-setuptools_scm${MODPY_FLAVOR}
+
+LIB_DEPENDS = archivers/lz4

-MODPY_VERSION = ${MODPY_DEFAULT_VERSION_3}
+TEST_DEPENDS = sysutils/py-psutil${MODPY_FLAVOR}

-TEST_DEPENDS += devel/py-tox${MODPY_FLAVOR} \
- devel/py-test${MODPY_FLAVOR} \
- devel/py-test-cov${MODPY_FLAVOR}
+FLAVORS = python3
+FLAVOR = python3

.include <bsd.port.mk>