X-Git-Url: http://drtracing.org/?a=blobdiff_plain;f=configure.ac;h=7f712ad7f2745b79499b0503737f7c07d49f3cf5;hb=313af7c991b598d3c6ef07aec86db799c47ac46f;hp=4d045308058db5858419a969cc978e1162bcacc4;hpb=d268885ad55f84ce6465992bc82b47282b296a4b;p=librseq.git

diff --git a/configure.ac b/configure.ac
index 4d04530..7f712ad 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,6 +1,6 @@
 dnl SPDX-License-Identifier: MIT
 dnl
-dnl Copyright (C) 2021 EfficiOS, Inc.
+dnl SPDX-FileCopyrightText: 2021 EfficiOS, Inc.
 dnl
 dnl Process this file with autoconf to produce a configure script.
 
@@ -80,6 +80,44 @@ AC_TYPE_SIZE_T
 AC_TYPE_UINT32_T
 AC_TYPE_UINT64_T
 
+# Detect warning flags supported by the C compiler and append them to
+# WARN_CFLAGS.
+m4_define([WARN_FLAGS_LIST], [ dnl
+  -Wall dnl
+  -Wextra dnl
+  -Wmissing-prototypes dnl
+  -Wmissing-declarations dnl
+  -Wnull-dereference dnl
+  -Wundef dnl
+  -Wshadow dnl
+  -Wjump-misses-init dnl
+  -Wsuggest-attribute=format dnl
+  -Wtautological-constant-out-of-range-compare dnl
+  -Wnested-externs dnl
+  -Wwrite-strings dnl
+  -Wformat=2 dnl
+  -Wstrict-aliasing dnl
+  -Wmissing-noreturn dnl
+  -Winit-self dnl
+  -Wduplicated-cond dnl
+  -Wduplicated-branches dnl
+  -Wlogical-op dnl
+  -Wredundant-decls dnl
+])
+
+# Pass -Werror as an extra flag during the test: this is needed to make the
+# -Wunknown-warning-option diagnostic fatal with clang.
+AC_LANG_PUSH([C])
+AX_APPEND_COMPILE_FLAGS([WARN_FLAGS_LIST], [WARN_CFLAGS], [-Werror])
+AC_LANG_POP([C])
+
+AC_LANG_PUSH([C++])
+AX_APPEND_COMPILE_FLAGS([WARN_FLAGS_LIST], [WARN_CXXFLAGS], [-Werror])
+AC_LANG_POP([C++])
+
+AE_IF_FEATURE_ENABLED([Werror], [WARN_CFLAGS="${WARN_CFLAGS} -Werror"])
+AE_IF_FEATURE_ENABLED([Werror], [WARN_CXXFLAGS="${WARN_CXXFLAGS} -Werror"])
+
 
 ##                     ##
 ## C++ compiler checks ##
@@ -100,10 +138,6 @@ AC_CHECK_HEADERS([ \
 	sys/time.h \
 ])
 
-AC_CHECK_HEADER([linux/rseq.h], [],
-	[AC_MSG_ERROR([Cannot find 'linux/rseq.h'.])
-])
-
 
 ##                 ##
 ## Programs checks ##
@@ -148,6 +182,17 @@ AC_CHECK_LIB([dl], [dlopen], [
 ])
 AC_SUBST(DL_LIBS)
 
+
+##                             ##
+## Optional features selection ##
+##                             ##
+
+# When given, add -Werror to WARN_CFLAGS and WARN_CXXFLAGS.
+# Disabled by default
+AE_FEATURE_DEFAULT_DISABLE
+AE_FEATURE([Werror], [Treat compiler warnings as errors.])
+
+
 ##                                             ##
 ## Substitute variables for use in Makefile.am ##
 ##                                             ##
@@ -161,10 +206,10 @@ AC_SUBST([RSEQ_LIBRARY_VERSION], [rseq_lib_version])
 AM_CPPFLAGS="-I\$(top_builddir)/include -I\$(top_srcdir)/include -include config.h"
 AC_SUBST(AM_CPPFLAGS)
 
-AM_CFLAGS="-Wall -Wextra $PTHREAD_CFLAGS"
+AM_CFLAGS="$WARN_CFLAGS $PTHREAD_CFLAGS"
 AC_SUBST(AM_CFLAGS)
 
-AM_CXXFLAGS="-Wall -Wextra $PTHREAD_CFLAGS"
+AM_CXXFLAGS="$WARN_CXXFLAGS $PTHREAD_CFLAGS"
 AC_SUBST(AM_CXXFLAGS)
 
 
@@ -181,6 +226,7 @@ AC_CONFIG_FILES([
 	src/librseq.pc
 	tests/Makefile
 	tests/utils/Makefile
+	tests/unit/Makefile
 ])
 
 AC_OUTPUT