[Buildroot] [PATCH 3/3] package/tpm2-tools: add option for enabling FAPI tools

[Arnout Vandecappelle]

On 01/01/2022 23:39, Erik Larsson wrote:
> Signed-off-by: Erik Larsson <who+buildroot at cnackers.org>
> ---
>   package/tpm2-tools/Config.in     | 11 +++++++++++
>   package/tpm2-tools/tpm2-tools.mk |  6 ++++++
>   2 files changed, 17 insertions(+)
> 
> diff --git a/package/tpm2-tools/Config.in b/package/tpm2-tools/Config.in
> index 1a6698570a..cb3eb2b0a5 100644
> --- a/package/tpm2-tools/Config.in
> +++ b/package/tpm2-tools/Config.in
> @@ -21,6 +21,17 @@ config BR2_PACKAGE_TPM2_TOOLS
>   
>   	  https://github.com/tpm2-software/tpm2-tools
>   
> +if BR2_PACKAGE_TPM2_TOOLS
> +
> +config BR2_PACKAGE_TPM2_TOOLS_FAPI
> +	bool "tpm2-tools fapi support"
> +	select BR2_PACKAGE_TPM2_TSS_FAPI
> +	help
> +	  This option enables Feature API (FAPI) support in tpm2-tools.
> +	  The FAPI tools are prefixed with tss2_.
> +
> +endif
> +
>   comment "tpm2-tools needs a glibc or musl toolchain w/ dynamic library, wchar"
>   	depends on BR2_STATIC_LIBS || !BR2_USE_WCHAR || \
>   		!(BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL)
> diff --git a/package/tpm2-tools/tpm2-tools.mk b/package/tpm2-tools/tpm2-tools.mk
> index df5fbbfff4..6894be195f 100644
> --- a/package/tpm2-tools/tpm2-tools.mk
> +++ b/package/tpm2-tools/tpm2-tools.mk
> @@ -22,4 +22,10 @@ TPM2_TOOLS_CONF_OPTS = --disable-hardening
>   # do not build man pages
>   TPM2_TOOLS_CONF_ENV += ac_cv_prog_PANDOC=''
>   
> +ifeq ($(BR2_PACKAGE_TPM2_TOOLS_FAPI),y)

  Does this optoin make a big difference in size? If not, we'd normally make it 
an automatic option, i.e.

ifeq ($(BR2_PACKAGE_TPM2_TSS_FAPI),y)

(and remove the Config.in option).

  Regards,
  Arnout

> +TPM2_TOOLS_CONF_OPTS += --enable-fapi
> +else
> +TPM2_TOOLS_CONF_OPTS += --disable-fapi
> +endif
> +
>   $(eval $(autotools-package))
>