[Buildroot] [PATCH 1/1] safeclib: fix build without wchar and host-doxygen

[Thomas Petazzoni <thomas.petazzoni@bootlin.com>]

Hello,

On Thu, 15 Mar 2018 20:37:11 +0100, Fabrice Fontaine wrote:

> diff --git a/package/safeclib/0001-Add-disable-doc-option.patch b/package/safeclib/0001-Add-disable-doc-option.patch
> new file mode 100644
> index 0000000000..67c48906f3
> --- /dev/null
> +++ b/package/safeclib/0001-Add-disable-doc-option.patch
> @@ -0,0 +1,58 @@
> +From 0d56b6327f6b652511c3e4c72382f4dc4d194e10 Mon Sep 17 00:00:00 2001
> +From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +Date: Thu, 15 Mar 2018 20:12:28 +0100
> +Subject: [PATCH] Add --disable-doc option
> +
> +This option allows the user to disable man pages even if pod2man and
> +doxygen programs are found
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Since it has been applied upstream now, I've added a reference to the
upstream commit here.

> ++AC_ARG_ENABLE(doc,
> ++	AS_HELP_STRING([--disable-doc],
> ++	               [disable documentation @<:@default=no@:>@]),
> ++	[case "${enableval}" in
> ++	yes) enable_doc=true ;;
> ++	no)  enable_doc=false ;;
> ++	*)   AC_MSG_ERROR([bad value ${enableval} for --enable-doc]) ;;

I haven't seen many configure.ac scripts that do this. Most simply do:

	enable_doc=${enableval}

and that's it.

But OK, it's been accepted upstream, so I don't care :)

Patch applied to master. Thanks!

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com