From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Bruno Thomsen <bruno.thomsen@gmail.com>, bth@kamstrup.com
Subject: Re: [ptxdist] [PATCH 3/5] host-squashfs-tools: add support for zstd contitional
Date: Fri, 17 Jul 2020 08:35:47 +0200 [thread overview]
Message-ID: <20200717063547.GF27485@pengutronix.de> (raw)
In-Reply-To: <20200716173805.7467-3-bruno.thomsen@gmail.com>
On Thu, Jul 16, 2020 at 07:38:03PM +0200, Bruno Thomsen wrote:
> Only enable host zstd package dependency when needed,
> so it doesn't build unnecessary packages.
>
> Signed-off-by: Bruno Thomsen <bruno.thomsen@gmail.com>
> ---
> rules/host-squashfs-tools.in | 1 +
> rules/host-squashfs-tools.make | 12 ++++++++++--
> 2 files changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/rules/host-squashfs-tools.in b/rules/host-squashfs-tools.in
> index 4ac2fa1b0..5ea45120e 100644
> --- a/rules/host-squashfs-tools.in
> +++ b/rules/host-squashfs-tools.in
> @@ -5,4 +5,5 @@ config HOST_SQUASHFS_TOOLS
> select HOST_ZLIB
> select HOST_LIBLZO
> select HOST_XZ
> + select HOST_ZSTD if IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_ZSTD
> default y if ALLYES
> diff --git a/rules/host-squashfs-tools.make b/rules/host-squashfs-tools.make
> index 39a6d11e1..45d69ba8b 100644
> --- a/rules/host-squashfs-tools.make
> +++ b/rules/host-squashfs-tools.make
> @@ -21,8 +21,16 @@ HOST_SQUASHFS_TOOLS_SUBDIR := squashfs-tools
> # Compile
> # ----------------------------------------------------------------------------
>
> -HOST_SQUASHFS_TOOLS_MAKE_OPT := LZO_SUPPORT=1 XZ_SUPPORT=1
> -HOST_SQUASHFS_TOOLS_MAKE_ENV := $(HOST_ENV) EXTRA_LDFLAGS="-L$(PTXDIST_SYSROOT_HOST)/lib/xz"
> +HOST_SQUASHFS_TOOLS_MAKE_OPT := \
> + LZO_SUPPORT=1 \
> + XZ_SUPPORT=1 \
> + ZSTD_SUPPORT=$(call ptx/ifdef, PTXCONF_IMAGE_ROOT_SQUASHFS_COMPRESSION_MODE_ZSTD,1,0)
> +
> +HOST_SQUASHFS_TOOLS_MAKE_ENV := \
> + $(HOST_ENV) \
> + EXTRA_LDFLAGS="-L$(PTXDIST_SYSROOT_HOST)/lib/xz -L$(PTXDIST_SYSROOT_HOST)/usr/local/lib64" \
> + CPPFLAGS="-I$(PTXDIST_SYSROOT_HOST)/usr/local/include"
The /usr/local paths should not be necessary with the fixed host-zstd
package.
Michael
> +
> HOST_SQUASHFS_TOOLS_MAKE_PAR := NO
> HOST_SQUASHFS_TOOLS_INSTALL_OPT = install INSTALL_DIR="$(HOST_SQUASHFS_TOOLS_PKGDIR)/sbin"
>
> --
> 2.26.2
>
>
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
next prev parent reply other threads:[~2020-07-17 6:35 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-16 17:38 [ptxdist] [PATCH 1/5] host-zstd: new package Bruno Thomsen
2020-07-16 17:38 ` [ptxdist] [PATCH 2/5] image-root-squashfs: add zstd compression option Bruno Thomsen
2020-07-16 17:38 ` [ptxdist] [PATCH 3/5] host-squashfs-tools: add support for zstd contitional Bruno Thomsen
2020-07-17 6:35 ` Michael Olbrich [this message]
2020-07-17 6:46 ` Bruno Thomsen
2020-07-16 17:38 ` [ptxdist] [PATCH 4/5] host-squashfs-tools: conditional select lzo and xz dependencies Bruno Thomsen
2020-07-16 17:38 ` [ptxdist] [PATCH 5/5] host-squashfs-tools: fix lz4 compression option Bruno Thomsen
2020-07-17 6:34 ` [ptxdist] [PATCH 1/5] host-zstd: new package 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=20200717063547.GF27485@pengutronix.de \
--to=m.olbrich@pengutronix.de \
--cc=bruno.thomsen@gmail.com \
--cc=bth@kamstrup.com \
--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