UPDATE zstd - enable lz4- and xz-support

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

UPDATE zstd - enable lz4- and xz-support

Björn Ketelaars
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 <[hidden email]>
+
 # 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}"