From: Jarkko Sakkinen via buildroot <buildroot@buildroot.org>
To: Fiona Klute <fiona.klute@gmx.de>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 3/3] package/swtpm: disable building manpages
Date: Thu, 15 Oct 2026 06:10:44 +0300 [thread overview]
Message-ID: <atBENLaaSiBfwORj@kernel.org> (raw)
In-Reply-To: <20261014164625.1474866-3-fiona.klute@gmx.de>
On Wed, Oct 14, 2026 at 06:46:25PM +0200, Fiona Klute wrote:
> From: "Fiona Klute (othermo GmbH)" <fiona.klute@gmx.de>
>
> Swtpm builds manpages using pod2man, and fails if it is not
> available. The patch adds support for --disable-docs, which Buildroot
> passes by default. Found while building in a container with minimal
> dependencies.
>
> Fixes: 87b9b126d23be1e95b3d2b82438f65e28f554a87
>
> Signed-off-by: Fiona Klute (othermo GmbH) <fiona.klute@gmx.de>
> ---
> ...ld-sys-Allow-disabling-manpage-build.patch | 63 +++++++++++++++++++
> 1 file changed, 63 insertions(+)
> create mode 100644 package/swtpm/0001-build-sys-Allow-disabling-manpage-build.patch
>
> diff --git a/package/swtpm/0001-build-sys-Allow-disabling-manpage-build.patch b/package/swtpm/0001-build-sys-Allow-disabling-manpage-build.patch
> new file mode 100644
> index 0000000000..c134b75c23
> --- /dev/null
> +++ b/package/swtpm/0001-build-sys-Allow-disabling-manpage-build.patch
> @@ -0,0 +1,63 @@
> +From 43eff71dc1673c252352c584e129e5bce312d173 Mon Sep 17 00:00:00 2001
> +From: Fiona Klute <fiona.klute@gmx.de>
> +Date: Wed, 14 Oct 2026 17:46:51 +0200
> +Subject: [PATCH] build-sys: Allow disabling manpage build
> +
> +In builds for embedded systems (e.g. Buildroot) manpages are not
> +needed, and building them adds a dependency on pod2man. Allow passing
> +"--disable-docs" to configure to disable building manpages.
> +
> +Signed-off-by: Fiona Klute <fiona.klute@gmx.de>
> +Upstream: https://github.com/stefanberger/swtpm/pull/1186
> +---
> + Makefile.am | 5 ++++-
> + configure.ac | 6 ++++++
> + 2 files changed, 10 insertions(+), 1 deletion(-)
> +
> +diff --git a/Makefile.am b/Makefile.am
> +index 5dfaa6b..f7d15fe 100644
> +--- a/Makefile.am
> ++++ b/Makefile.am
> +@@ -6,11 +6,14 @@
> +
> + SUBDIRS = \
> + include \
> +- man \
> + samples \
> + src \
> + tests
> +
> ++if ENABLE_DOCS
> ++SUBDIRS += man
> ++endif # ENABLE_DOCS
> ++
> + ACLOCAL_AMFLAGS = -I m4
> +
> + EXTRA_DIST = \
> +diff --git a/configure.ac b/configure.ac
> +index 188b1a2..7156a21 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -509,6 +509,11 @@ if test "x$enable_sanitizers" = "xyes"; then
> + CFLAGS="$save_CFLAGS"
> + fi
> +
> ++AC_ARG_ENABLE([docs],
> ++ [AS_HELP_STRING([--disable-docs], [Disable building documentation])],,
> ++ [enable_docs=yes])
> ++AM_CONDITIONAL(ENABLE_DOCS, [test "$enable_docs" = "yes"])
> ++
> + AC_ARG_WITH([tss-user],
> + AS_HELP_STRING([--with-tss-user=TSS_USER],[The tss user to use]),
> + [TSS_USER="$withval"],
> +@@ -662,6 +667,7 @@ printf "TPM 2 profile support : yes\n"
> + echo
> + echo "Version to build : $PACKAGE_VERSION"
> + echo "Crypto library : $cryptolib"
> ++echo "Documentation : $enable_docs"
> + echo
> + echo " MY_CFLAGS = $MY_CFLAGS"
> + echo " HARDENING_CFLAGS = $HARDENING_CFLAGS"
> +--
> +2.55.0
> +
> --
> 2.55.0
>
Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org>
Br, Jarkko
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2026-10-15 3:10 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-14 16:46 [Buildroot] [PATCH 1/3] package/libtpms: fix typo in HOST_LIBTPMS_CONF_OPTS Fiona Klute via buildroot
2026-10-14 16:46 ` [Buildroot] [PATCH 2/3] package/libtpms: disable building manpages Fiona Klute via buildroot
2026-10-15 3:09 ` Jarkko Sakkinen via buildroot
2026-10-14 16:46 ` [Buildroot] [PATCH 3/3] package/swtpm: " Fiona Klute via buildroot
2026-10-15 3:10 ` Jarkko Sakkinen via buildroot [this message]
2026-10-14 20:38 ` [Buildroot] [PATCH 1/3] package/libtpms: fix typo in HOST_LIBTPMS_CONF_OPTS Jarkko Sakkinen via buildroot
2026-10-14 20:52 ` Fiona Klute via buildroot
2026-10-15 3:02 ` Jarkko Sakkinen via buildroot
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=atBENLaaSiBfwORj@kernel.org \
--to=buildroot@buildroot.org \
--cc=fiona.klute@gmx.de \
--cc=jarkko@kernel.org \
/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