From 9ad74c455a1f58d5e8d84d82d3cbba9cb27360e9 Mon Sep 17 00:00:00 2001 From: Adam Majer Date: Mon, 10 Apr 2023 16:22:46 +0200 Subject: [PATCH] Drop C++ references and obsolete C standards (#513) With no C++ compiler needed, there is no need to set the CXXFLAGS or log it during configure. Furthermore, forcing outdated C11 standard is not a good idea considering compilers have moved on. We should not limit our feature set, including compiler warnings, to old standards. --- configure.ac | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index 844025626..610f15b67 100644 --- a/configure.ac +++ b/configure.ac @@ -379,11 +379,9 @@ AS_IF([test x"$enable_werror" = "xyes"], [ ]) AS_IF([test x"$GCC" = "xyes"], [ # Be tough with warnings and produce less careless code - CFLAGS="$CFLAGS -Wall -std=gnu11" - CXXFLAGS="$CXXFLAGS -Wall " # -Weffc++" # TODO: enable when it does not print 1MB of warnings + CFLAGS="$CFLAGS -Wall" ]) CFLAGS="$CFLAGS -D_GNU_SOURCE" -CXXFLAGS="$CXXFLAGS -D_GNU_SOURCE" # ========================================================= # Select a different shell instead of the default /bin/bash @@ -411,7 +409,6 @@ AC_MSG_NOTICE([ ============================================================================== Environment settings: CFLAGS: ${CFLAGS} - CXXFLAGS: ${CXXFLAGS} LDFLAGS: ${LDFLAGS} Build configuration: cups-config: ${with_cups_config} -- 2.47.3