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>

Reply via email to