From: Jarkko Sakkinen via buildroot <buildroot@buildroot.org>
To: Fiona Klute <fiona.klute@gmx.de>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 2/3] package/libtpms: disable building manpages
Date: Thu, 15 Oct 2026 06:09:56 +0300 [thread overview]
Message-ID: <atBEBB1x4WI-GRh8@kernel.org> (raw)
In-Reply-To: <20261014164625.1474866-2-fiona.klute@gmx.de>
On Wed, Oct 14, 2026 at 06:46:24PM +0200, Fiona Klute wrote:
> From: "Fiona Klute (othermo GmbH)" <fiona.klute@gmx.de>
>
> Libtpms 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: b2eec8c6265a38cdba1afadf7f60e0fd00f3b448
>
> Signed-off-by: Fiona Klute (othermo GmbH) <fiona.klute@gmx.de>
> ---
> ...ld-sys-Allow-disabling-manpage-build.patch | 61 +++++++++++++++++++
> 1 file changed, 61 insertions(+)
> create mode 100644 package/libtpms/0002-build-sys-Allow-disabling-manpage-build.patch
>
> diff --git a/package/libtpms/0002-build-sys-Allow-disabling-manpage-build.patch b/package/libtpms/0002-build-sys-Allow-disabling-manpage-build.patch
> new file mode 100644
> index 0000000000..20849db49d
> --- /dev/null
> +++ b/package/libtpms/0002-build-sys-Allow-disabling-manpage-build.patch
> @@ -0,0 +1,61 @@
> +From 6e688d2bd1e6e1545fdb25bcfeee28f78240cbfb Mon Sep 17 00:00:00 2001
> +From: Fiona Klute <fiona.klute@gmx.de>
> +Date: Wed, 14 Oct 2026 17:31:09 +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/libtpms/pull/641
> +---
> + Makefile.am | 5 ++++-
> + configure.ac | 6 ++++++
> + 2 files changed, 10 insertions(+), 1 deletion(-)
> +
> +diff --git a/Makefile.am b/Makefile.am
> +index 03ed7e65..9d0ea053 100644
> +--- a/Makefile.am
> ++++ b/Makefile.am
> +@@ -23,10 +23,13 @@
> +
> + SUBDIRS = \
> + include \
> +- man \
> + 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 b19db106..0fc928c8 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -282,6 +282,11 @@ AC_ARG_ENABLE([test-coverage],
> + ]
> + )
> +
> ++AC_ARG_ENABLE([docs],
> ++ [AS_HELP_STRING([--disable-docs], [Disable building documentation])],,
> ++ [enable_docs=yes])
> ++AM_CONDITIONAL(ENABLE_DOCS, [test "$enable_docs" = "yes"])
> ++
> + LT_INIT
> + AC_PROG_CC
> + AC_PROG_CXX
> +@@ -392,5 +397,6 @@ echo "Use openssl crypto for : $use_openssl_functions_for"
> + echo "Test coverage : $enable_test_coverage"
> + echo "Static build : $enable_static"
> + echo "Statically linked tests : $enable_static_tests"
> ++echo "Build documentation : $enable_docs"
> + echo
> + echo
> +--
> +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 [this message]
2026-10-14 16:46 ` [Buildroot] [PATCH 3/3] package/swtpm: " Fiona Klute via buildroot
2026-10-15 3:10 ` Jarkko Sakkinen via buildroot
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=atBEBB1x4WI-GRh8@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.