zstd offers support for lz4 and xz. Diff below enables this support for the
version in ports. Please note that support for xz has been dropped in
a previous revision [0]. I do not know why.

Output of 'make test' looks ok. Running zstd with --format=lz4, or
--format=xz works as promised.

I would like to take maintainership as this port is a dependency of another
port that I maintain.

Comments?

[0]
http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/archivers/zstd/Makefile.diff?r1=1.11&r2=1.12&f=h


diff --git archivers/zstd/Makefile archivers/zstd/Makefile
index 945204f89e9..3b41359437a 100644
--- archivers/zstd/Makefile
+++ archivers/zstd/Makefile
@@ -5,25 +5,35 @@ COMMENT =             zstandard fast real-time compression 
algorithm
 GH_ACCOUNT =           facebook
 GH_PROJECT =           zstd
 GH_TAGNAME =           v1.3.3
+REVISION =             0
 SHARED_LIBS =          zstd    0.0     #1.3.3
 
 CATEGORIES =           archivers
 
 HOMEPAGE =             https://facebook.github.io/zstd/
 
+MAINTAINER =           Bjorn Ketelaars <bjorn.ketela...@hydroxide.nl>
+
 # zstd doesn't use known patented code or algorithms, however the project
 # includes an additional patent grant in the file PATENTS.
 # https://github.com/facebook/zstd/issues/335
 # BSD 3 clauses + GPLv2.
 PERMIT_PACKAGE_CDROM = Yes
 
-WANTLIB =              c pthread z
+WANTLIB =              c lz4 lzma pthread  z
+
+LIB_DEPENDS =          archivers/lz4 \
+                       archivers/xz
 
 COMPILER =             base-clang ports-clang ports-gcc
 COMPILER_LANGS =       c
 
-MAKE_ENV =             CC="${CC}" CXX="${CXX}" \
-                       CFLAGS="${CFLAGS}" CXXFLAGS="${CXXFLAGS}" \
+MAKE_ENV =             CC="${CC}" \
+                       CFLAGS="${CFLAGS}" \
+                       CPPFLAGS="-I${LOCALBASE}/include" \
+                       CXX="${CXX}" \
+                       CXXFLAGS="${CXXFLAGS}" \
+                       LDFLAGS="-L${LOCALBASE}/lib" \
                        LIBzstd_VERSION="${LIBzstd_VERSION}"
 FAKE_FLAGS =           PREFIX="${PREFIX}"
 

Reply via email to