From 813e7bdac1308a39be89eba7eb02063970f795b3 Mon Sep 17 00:00:00 2001
From: Jim Barlow <jim@purplerock.ca>
Date: Thu, 1 Jan 2015 15:03:03 -0800
Subject: Change comments to remark on script not being needed for clang >= 5.1

---
 configure.ac                         |  2 +-
 scripts/wrap-compiler-for-flag-check | 10 +++++++---
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/configure.ac b/configure.ac
index eee0221..18321bc 100644
--- a/configure.ac
+++ b/configure.ac
@@ -80,7 +80,7 @@ AC_COMPILE_IFELSE([AC_LANG_PROGRAM([], [[
 AC_MSG_RESULT([$CLANG])
 AS_IF([test "x$CLANG" == "xyes"], [CLANG_FLAGS=-Qunused-arguments])
 
-# We want to check for compiler flag support, but there is no way to make
+# We want to check for compiler flag support. Prior to clang v5.1, there was no way to make
 # clang's "argument unused" warning fatal.  So we invoke the compiler through a
 # wrapper script that greps for this message.
 saved_CC="$CC"
diff --git a/scripts/wrap-compiler-for-flag-check b/scripts/wrap-compiler-for-flag-check
index 6fa77f0..ee1100c 100755
--- a/scripts/wrap-compiler-for-flag-check
+++ b/scripts/wrap-compiler-for-flag-check
@@ -3,12 +3,16 @@
 # From kmcallister: 
 # https://github.com/kmcallister/autoharden/blob/efaf5a16612589808c276a11536ea9a47071f74b/scripts/wrap-compiler-for-flag-check
 
-# There is no way to make clang's "argument unused" warning fatal.  So when
-# configure checks for supported flags, it runs $CC, $CXX, $LD via this
-# wrapper.
+# Prior to clang v5.1, there was no way to make
+# clang's "argument unused" warning fatal.  This
+# wrapper script that greps for this warning message. Newer clang's have no issues.
 #
 # Ideally the search string would also include 'clang: ' but this output might
 # depend on clang's argv[0].
+#
+# This file is in the public domain.
+set -o errexit
+set -o nounset
 
 if out=`"$@" 2>&1`; then
   echo "$out"
-- 
cgit v1.2.3-55-g6feb