mirror of
https://gitlab.com/buildroot.org/buildroot.git
synced 2026-08-01 21:23:51 -09:00
We're about to bump abseil to a version >= 20250512.02, which requires C++17, meaning that protobuf/grpc will also C++17. This in turn means that the grpc support in collectd will also need C++17. So just like our current patch 0002-configure.ac-fix-grpc-build.patch updates the collectd logic from C++11 to C++14, this new patch (accepted upstream) updates the collectd logic again from C++14 to C++17. Signed-off-by: Michael Nosthoff <buildroot@heine.tech> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
74 lines
3.0 KiB
Diff
74 lines
3.0 KiB
Diff
From 42b2ede4dadcf88293e408bcdf5645995b61b148 Mon Sep 17 00:00:00 2001
|
|
From: Michael Nosthoff <buildroot@heine.tech>
|
|
Date: Wed, 29 Oct 2025 16:48:20 +0100
|
|
Subject: [PATCH] configure.ac: fix grpc build
|
|
|
|
Similar to the previous "configure.ac: fix grpc build" commit:
|
|
Google bumped it's compiler requirements again [1], hence the c++ version needs to be adjusted to c++17 to be able to build with the latest grpc versions.
|
|
|
|
[1] https://github.com/google/oss-policies-info/blob/a5e3ed43d45ad615a20201cea8f7282f595555da/foundational-cxx-support-matrix.md
|
|
|
|
Signed-off-by: Michael Nosthoff <buildroot@heine.tech>
|
|
Upstream: https://github.com/collectd/collectd/commit/42b2ede4dadcf88293e408bcdf5645995b61b148
|
|
---
|
|
configure.ac | 16 ++++++++--------
|
|
1 file changed, 8 insertions(+), 8 deletions(-)
|
|
|
|
diff --git a/configure.ac b/configure.ac
|
|
index 8aa03fce63..eb6cbbc5d5 100644
|
|
--- a/configure.ac
|
|
+++ b/configure.ac
|
|
@@ -2844,20 +2844,20 @@ PKG_CHECK_MODULES([GRPCPP], [grpc++],
|
|
fi
|
|
|
|
if test "x$withval" != "xno"; then
|
|
- AC_MSG_CHECKING([whether $CXX accepts -std=c++14])
|
|
- if test_cxx_flags -std=c++14; then
|
|
+ AC_MSG_CHECKING([whether $CXX accepts -std=c++17])
|
|
+ if test_cxx_flags -std=c++17; then
|
|
AC_MSG_RESULT([yes])
|
|
else
|
|
AC_MSG_RESULT([no])
|
|
- with_libgrpcpp="no (requires C++14 support)"
|
|
- with_libprotobuf="no (<absl/base/policy_checks.h> requires C++14 support)"
|
|
+ with_libgrpcpp="no (requires C++17 support)"
|
|
+ with_libprotobuf="no (<absl/base/policy_checks.h> requires C++17 support)"
|
|
fi
|
|
fi
|
|
|
|
if test "x$with_libgrpcpp" = "xyes"; then
|
|
AC_LANG_PUSH(C++)
|
|
SAVE_CPPFLAGS="$CPPFLAGS"
|
|
- CPPFLAGS="-std=c++14 $with_libgrpcpp_cppflags $GRPCPP_CFLAGS $CPPFLAGS"
|
|
+ CPPFLAGS="-std=c++17 $with_libgrpcpp_cppflags $GRPCPP_CFLAGS $CPPFLAGS"
|
|
AC_CHECK_HEADERS([grpc++/grpc++.h],
|
|
[with_libgrpcpp="yes"],
|
|
[with_libgrpcpp="no (<grpc++/grpc++.h> not found)"]
|
|
@@ -2871,7 +2871,7 @@ if test "x$with_libgrpcpp" = "xyes"; then
|
|
SAVE_CPPFLAGS="$CPPFLAGS"
|
|
SAVE_LDFLAGS="$LDFLAGS"
|
|
SAVE_LIBS="$LIBS"
|
|
- CPPFLAGS="-std=c++14 $with_libgrpcpp_cppflags $GRPCPP_CFLAGS $CPPFLAGS"
|
|
+ CPPFLAGS="-std=c++17 $with_libgrpcpp_cppflags $GRPCPP_CFLAGS $CPPFLAGS"
|
|
LDFLAGS="$with_libgrpcpp_ldflags"
|
|
if test "x$GRPCPP_LIBS" = "x"; then
|
|
LIBS="-lgrpc++"
|
|
@@ -2899,7 +2899,7 @@ if test "x$with_libgrpcpp" = "xyes"; then
|
|
AC_LANG_POP(C++)
|
|
fi
|
|
|
|
-BUILD_WITH_LIBGRPCPP_CPPFLAGS="-std=c++14 $with_libgrpcpp_cppflags $GRPCPP_CFLAGS"
|
|
+BUILD_WITH_LIBGRPCPP_CPPFLAGS="-std=c++17 $with_libgrpcpp_cppflags $GRPCPP_CFLAGS"
|
|
BUILD_WITH_LIBGRPCPP_LDFLAGS="$with_libgrpcpp_ldflags"
|
|
BUILD_WITH_LIBGRPCPP_LIBS="$GRPCPP_LIBS"
|
|
AC_SUBST([BUILD_WITH_LIBGRPCPP_CPPFLAGS])
|
|
@@ -4843,7 +4843,7 @@ if test "x$withval" != "xno"; then
|
|
AC_CHECK_LIB([protobuf], [main],
|
|
[
|
|
SAVE_CPPFLAGS="$CPPFLAGS"
|
|
- CPPFLAGS="-std=c++14 $with_libprotobuf_cppflags $PROTOBUF_CFLAGS"
|
|
+ CPPFLAGS="-std=c++17 $with_libprotobuf_cppflags $PROTOBUF_CFLAGS"
|
|
if test "x$PROTOBUF_LIBS" = "x"
|
|
then
|
|
PROTOBUF_LIBS="-lprotobuf"
|