mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Roland Hieber <rhi@pengutronix.de>
Subject: Re: [ptxdist] [PATCH] host-mtd-utils: build with crypto support
Date: Wed, 16 Jun 2021 13:44:39 +0200	[thread overview]
Message-ID: <20210616114439.GK839947@pengutronix.de> (raw)
In-Reply-To: <20210612204835.7372-1-rhi@pengutronix.de>

On Sat, Jun 12, 2021 at 10:48:35PM +0200, Roland Hieber wrote:
> --with-crypto is needed e.g. for UBIFS Authentication support in
> mkfs.ubifs, and the additional dependencies are negligible (and probably
> even required for other host tools).

That's not quite true. The problem with the host-kernel-header is that the
default version comes from kernel version. So whenever the kernel version
changes, anything depending on kernel headers is rebuilt.
That's something I'd like to avoid.

Also, for the UBIFS authentication, you need a special image anyways,
right? So selecting HOST_MTD_UTILS_CRYPTO here would not be a problem.

> Signed-off-by: Roland Hieber <rhi@pengutronix.de>
> ---
>  rules/host-mtd-utils.in   | 2 ++
>  rules/host-mtd-utils.make | 2 +-
>  2 files changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/rules/host-mtd-utils.in b/rules/host-mtd-utils.in
> index 4bdd554d94b1..25fd654e2b49 100644
> --- a/rules/host-mtd-utils.in
> +++ b/rules/host-mtd-utils.in
> @@ -6,4 +6,6 @@ config HOST_MTD_UTILS
>  	select HOST_LIBLZO
>  	select HOST_LIBUUID
>  	select HOST_ZLIB
> +	select HOST_KERNEL_HEADER

Why is this needed anyways? What is so new in the kernel headers that it's
not available in most distributions?

Michael

> +	select HOST_OPENSSL
>  	default y if ALLYES
> diff --git a/rules/host-mtd-utils.make b/rules/host-mtd-utils.make
> index f49a08261781..20f30a7c7f42 100644
> --- a/rules/host-mtd-utils.make
> +++ b/rules/host-mtd-utils.make
> @@ -32,6 +32,6 @@ HOST_MTD_UTILS_CONF_OPT		:= \
>  	--with-lzo \
>  	--without-zstd \
>  	--without-selinux \
> -	--without-crypto
> +	--with-crypto
>  
>  # vim: syntax=make
> -- 
> 2.29.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


  reply	other threads:[~2021-06-16 11:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-12 20:48 Roland Hieber
2021-06-16 11:44 ` Michael Olbrich [this message]
2021-06-16 12:51   ` Roland Hieber
2021-06-16 13:12     ` 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=20210616114439.GK839947@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    --cc=rhi@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