mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH] host-mtd-utils: build with crypto support
@ 2021-06-12 20:48 Roland Hieber
  2021-06-16 11:44 ` Michael Olbrich
  0 siblings, 1 reply; 4+ messages in thread
From: Roland Hieber @ 2021-06-12 20:48 UTC (permalink / raw)
  To: ptxdist; +Cc: Roland Hieber

--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).

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
+	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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] [PATCH] host-mtd-utils: build with crypto support
  2021-06-12 20:48 [ptxdist] [PATCH] host-mtd-utils: build with crypto support Roland Hieber
@ 2021-06-16 11:44 ` Michael Olbrich
  2021-06-16 12:51   ` Roland Hieber
  0 siblings, 1 reply; 4+ messages in thread
From: Michael Olbrich @ 2021-06-16 11:44 UTC (permalink / raw)
  To: ptxdist; +Cc: Roland Hieber

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] [PATCH] host-mtd-utils: build with crypto support
  2021-06-16 11:44 ` Michael Olbrich
@ 2021-06-16 12:51   ` Roland Hieber
  2021-06-16 13:12     ` Michael Olbrich
  0 siblings, 1 reply; 4+ messages in thread
From: Roland Hieber @ 2021-06-16 12:51 UTC (permalink / raw)
  To: ptxdist

On Wed, Jun 16, 2021 at 01:44:39PM +0200, Michael Olbrich wrote:
> 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.

Ah right, a separate option for crypto support is also what I had
before, so I'll reintroduce it.

> 
> > 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?

Hmmm yes, it might be that we selected the kernel headers because we
needed to support a few old Ubuntus in the BSP. I can try without.

 - Roland

> 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 |
> 

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] [PATCH] host-mtd-utils: build with crypto support
  2021-06-16 12:51   ` Roland Hieber
@ 2021-06-16 13:12     ` Michael Olbrich
  0 siblings, 0 replies; 4+ messages in thread
From: Michael Olbrich @ 2021-06-16 13:12 UTC (permalink / raw)
  To: ptxdist

On Wed, Jun 16, 2021 at 02:51:59PM +0200, Roland Hieber wrote:
> On Wed, Jun 16, 2021 at 01:44:39PM +0200, Michael Olbrich wrote:
> > 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.
> 
> Ah right, a separate option for crypto support is also what I had
> before, so I'll reintroduce it.
> 
> > 
> > > 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?
> 
> Hmmm yes, it might be that we selected the kernel headers because we
> needed to support a few old Ubuntus in the BSP. I can try without.

Add a comment if it is really needed. So we know why this is there.

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 |
> > 
> 
> -- 
> Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
> Steuerwalder Str. 21                     | https://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
> 

-- 
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


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-06-16 13:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-12 20:48 [ptxdist] [PATCH] host-mtd-utils: build with crypto support Roland Hieber
2021-06-16 11:44 ` Michael Olbrich
2021-06-16 12:51   ` Roland Hieber
2021-06-16 13:12     ` Michael Olbrich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox