Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Jarkko Sakkinen" <jarkko@kernel.org>
To: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: buildroot@buildroot.org, Stefan Berger <stefanb@linux.ibm.com>
Subject: Re: [Buildroot] [PATCH v7 3/3] package/swtpm: add host package
Date: Thu, 06 Jun 2024 14:49:05 +0300	[thread overview]
Message-ID: <D1SWT4PQ5SRG.3CIZII0XWVO0O@kernel.org> (raw)
In-Reply-To: <ZmGBnfkfL4h2SDac@landeda>

On Thu Jun 6, 2024 at 12:30 PM EEST, Yann E. MORIN wrote:
> Jarkko, All,
>
> On 2024-06-06 02:35 +0300, Jarkko Sakkinen spake thusly:
> > The SWTPM package provides TPM emulators with different front-end
> > interfaces to libtpms. TPM emulators provide socket interfaces (TCP/IP and
> > Unix) and the Linux CUSE interface for the creation of multiple native
> > /dev/vtpm* devices.
> > 
> > Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org>
> > ---
> [--SNIP--]
> > diff --git a/package/swtpm/swtpm.mk b/package/swtpm/swtpm.mk
> > new file mode 100644
> > index 0000000000..b05a248c03
> > --- /dev/null
> > +++ b/package/swtpm/swtpm.mk
> > @@ -0,0 +1,23 @@
> > +################################################################################
> > +#
> > +# swtpm
> > +#
> > +################################################################################
> > +
> > +SWTPM_VERSION = 0.8.2
> > +SWTPM_SITE = $(call github,stefanberger,swtpm,v$(SWTPM_VERSION))
> > +SWTPM_LICENSE = BSD-4-Clause
> > +SWTPM_LICENSE_FILES = LICENSE
> > +
> > +# Required because a plain Git clone is used:
> > +HOST_SWTPM_AUTORECONF = YES
> > +HOST_SWTPM_DEPENDENCIES = \
> > +	host-json-glib \
> > +	host-libopenssl \
> > +	host-libtasn1 \
> > +	host-libtool \
> > +	host-libtpms \
> > +	host-pkgconf
> > +HOST_SWTPM_CONF_OPTS = --with-openssl --without-seccomp
> > +
> > +$(eval $(host-autotools-package))
>
>     $ make defconfig
>     $ make host-swtpm
>     [...]
>     configure: error: expect is required: expect package
>
> It is only needed for tests, so I disabled those.
>
> I took that opportunity to also reorganise the variables in a more
> logical manner: first, depenencies, then autoreconf, finally configure
> opts.
>
> Applied to next, thanks.

Thanks a lot for addressing those issues!

With systemd I'll test if just those fixes without version upgrade
would take of it because that would be a better order. And also
split into two patches.

I.e. thes transcripts:

Filesystem found in kernel header but not in filesystems-gperf.gperf: BCACHEFS_SUPER_MAGIC                                                    
Filesystem found in kernel header but not in filesystems-gperf.gperf: PID_FS_MAGIC                                                            

Is this anywhere near how the upstream references could be tagged?

Upstream: ed01b92e1c92 ("basic: add PIDFS magic (#31709)")
Upstream: 201148b08de6 ("basic: Add BCACHEFS magic")

I did not look into any bug database just plain scavenged the superblock
keywors for systemd's commit log :-)

Probably for a project like BuildRoot there must be really something
actually enforocing the upgrade :-)

>
> Regards,
> Yann E. MORIN.

BR, Jarkko
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2024-06-06 11:49 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-05 23:35 [Buildroot] [PATCH v7 0/3] swtpm and libtpms host packages Jarkko Sakkinen
2024-06-05 23:35 ` [Buildroot] [PATCH v7 1/3] package/json-glib: add host build Jarkko Sakkinen
2024-06-06  9:31   ` Yann E. MORIN
2024-06-05 23:35 ` [Buildroot] [PATCH v7 2/3] package/libtpms: add host package Jarkko Sakkinen
2024-06-05 23:35 ` [Buildroot] [PATCH v7 3/3] package/swtpm: " Jarkko Sakkinen
2024-06-06  9:30   ` Yann E. MORIN
2024-06-06 11:49     ` Jarkko Sakkinen [this message]
2024-06-06 11:54       ` Yann E. MORIN
2024-06-06  9:26 ` [Buildroot] [PATCH v7 0/3] swtpm and libtpms host packages Yann E. MORIN

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=D1SWT4PQ5SRG.3CIZII0XWVO0O@kernel.org \
    --to=jarkko@kernel.org \
    --cc=buildroot@buildroot.org \
    --cc=stefanb@linux.ibm.com \
    --cc=yann.morin.1998@free.fr \
    /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