[Buildroot] [PATCH 1/1] package/sdbusplus: needs C++17

[John Faith]

> From: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> Sent: Thursday, March 26, 2020 1:38 PM
> To: buildroot at buildroot.org <buildroot at buildroot.org>
> Cc: John Faith <jfaith at impinj.com>; Fabrice Fontaine <fontaine.fabrice at gmail.com>
> Subject: [PATCH 1/1] package/sdbusplus: needs C++17
> 
> sdbusplus needs C++17 since
> https://github.com/openbmc/sdbusplus/commit/ca46b9d2e958d1e4b392265f656f782899c2de1d
> 
> This is not needed for host-sdbusplus as libsdbusplus is disabled on
> host through --disable-libsdbusplus
> 
> Fixes:
> - 
> http://autobuild.buildroot.org/results/65a4652e5b419f03ace109c4b3738e9b997cdabe
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>

Reviewed-by: John Faith <jfaith at impinj.com>

Thanks for fixing this!
,
John

> ---
> package/sdbusplus/Config.in | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/package/sdbusplus/Config.in b/package/sdbusplus/Config.in
> index e278d17156..c66203c463 100644
> --- a/package/sdbusplus/Config.in
> +++ b/package/sdbusplus/Config.in
> @@ -2,6 +2,7 @@ config BR2_PACKAGE_SDBUSPLUS
> bool "sdbusplus"
> depends on BR2_INSTALL_LIBSTDCPP
> depends on BR2_PACKAGE_SYSTEMD
> + depends on BR2_TOOLCHAIN_GCC_AT_LEAST_7 # C++17
> help
> A C++ library for sd-bus with a bindings and doc generator.
> 
> @@ -10,5 +11,6 @@ config BR2_PACKAGE_SDBUSPLUS
> 
> https://github.com/openbmc/sdbusplus
> 
> -comment "sdbusplus needs systemd and a toolchain w/ C++"
> - depends on !BR2_PACKAGE_SYSTEMD || !BR2_INSTALL_LIBSTDCPP
> +comment "sdbusplus needs systemd and a toolchain w/ C++, gcc >= 7"
> + depends on !BR2_PACKAGE_SYSTEMD || !BR2_INSTALL_LIBSTDCPP || \
> + !BR2_TOOLCHAIN_GCC_AT_LEAST_7
> -- 
> 2.25.1