mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Marco Felsch <m.felsch@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: [ptxdist] [PATCH] zstd: remove libzstd targetinstall option
Date: Wed, 12 Aug 2020 17:27:03 +0200	[thread overview]
Message-ID: <20200812152703.7334-1-m.felsch@pengutronix.de> (raw)

Drop useless ZSTD_LIBZSTD selection. The main purpose of the zstd
package is to install the libzstd. Explicit selecting the zstdlib is
not intuitive and causes errors if someone assumes that this lib gets
installed just by selection the zstd package.

Link the zstd helper against the shared libzstd while on it.

Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
---
 rules/mesalib.in        | 1 -
 rules/squashfs-tools.in | 1 -
 rules/zstd.in           | 7 -------
 rules/zstd.make         | 5 ++---
 4 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/rules/mesalib.in b/rules/mesalib.in
index 4d87d0e61..df0da381c 100644
--- a/rules/mesalib.in
+++ b/rules/mesalib.in
@@ -31,7 +31,6 @@ menuconfig MESALIB
 	select EXPAT
 	select ZLIB
 	select ZSTD			if MESALIB_ZSTD
-	select ZSTD_LIBZSTD		if MESALIB_ZSTD
 	help
 	  Mesa is a 3-D graphics library with an API which is very
 	  similar to that of OpenGL. See the project homepage for
diff --git a/rules/squashfs-tools.in b/rules/squashfs-tools.in
index 9e4f48550..cbd1f324b 100644
--- a/rules/squashfs-tools.in
+++ b/rules/squashfs-tools.in
@@ -7,7 +7,6 @@ menuconfig SQUASHFS_TOOLS
 	select GCCLIBS_GCC_S
 	select ZLIB
 	select ZSTD		if SQUASHFS_TOOLS_ZSTD_SUPPORT
-	select ZSTD_LIBZSTD	if SQUASHFS_TOOLS_ZSTD_SUPPORT
 	prompt "squashfs-tools                "
 	help
 	  Squashfs is a highly compressed read-only filesystem for Linux.
diff --git a/rules/zstd.in b/rules/zstd.in
index 3010ebda3..b855c4e3a 100644
--- a/rules/zstd.in
+++ b/rules/zstd.in
@@ -15,23 +15,16 @@ menuconfig ZSTD
 
 if ZSTD
 
-config ZSTD_LIBZSTD
-	bool
-	prompt "install zstd library"
-
 config ZSTD_ZSTD
 	bool
-	select ZSTD_LIBZSTD
 	prompt "install zstd tool"
 
 config ZSTD_ZSTDGREP
 	bool
-	select ZSTD_LIBZSTD
 	prompt "install zstdgrep"
 
 config ZSTD_ZSTDLESS
 	bool
-	select ZSTD_LIBZSTD
 	prompt "install zstdless"
 
 endif
diff --git a/rules/zstd.make b/rules/zstd.make
index 670fba739..3dfcf1d6d 100644
--- a/rules/zstd.make
+++ b/rules/zstd.make
@@ -39,7 +39,7 @@ ZSTD_CONF_OPT	:= \
 	-DZSTD_BUILD_CONTRIB=OFF \
 	-DZSTD_BUILD_TESTS=OFF \
 	-DZSTD_USE_STATIC_RUNTIME=OFF \
-	-DZSTD_PROGRAMS_LINK_SHARED=OFF \
+	-DZSTD_PROGRAMS_LINK_SHARED=ON \
 	-DZSTD_BUILD_STATIC=ON \
 	-DZSTD_BUILD_SHARED=ON \
 	-DZSTD_ZLIB_SUPPORT=OFF \
@@ -59,9 +59,8 @@ $(STATEDIR)/zstd.targetinstall:
 	@$(call install_fixup, zstd, AUTHOR, "Florian Faber <faber@faberman.de>")
 	@$(call install_fixup, zstd, DESCRIPTION, missing)
 
-ifdef PTXCONF_ZSTD_LIBZSTD
 	@$(call install_lib, zstd, 0, 0, 0644, libzstd)
-endif
+
 ifdef PTXCONF_ZSTD_ZSTD
 	@$(call install_copy, zstd, 0, 0, 0755, -, /usr/bin/zstd)
 	@$(call install_link, zstd, zstd, /usr/bin/zstdcat)
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

             reply	other threads:[~2020-08-12 15:27 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-12 15:27 Marco Felsch [this message]
2020-08-17  6:18 ` [ptxdist] [APPLIED] " Michael Olbrich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200812152703.7334-1-m.felsch@pengutronix.de \
    --to=m.felsch@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox