|
From: <sv...@va...> - 2015-06-05 21:19:15
|
Author: florian
Date: Fri Jun 5 22:19:06 2015
New Revision: 15323
Log:
clang, as opposed to gcc, does not terminate with a non-zero return code
in case an unrecognised command line option is encountered. configure.ac
however was assuming just that which led to compile time warnings later on.
Add -Werror to the configure bits to make clang behave like gcc in this
regard. Fixes BZ #348565.
Modified:
trunk/NEWS
trunk/configure.ac
Modified: trunk/NEWS
==============================================================================
--- trunk/NEWS (original)
+++ trunk/NEWS Fri Jun 5 22:19:06 2015
@@ -226,8 +226,9 @@
347978 Remove bash dependencies where not needed
348102 Patch updating v4l2 API support
348247 jno jumps wrongly when overflow is not set
+348565 Fix detection of command line option availability for clang
348728 Fix broken check for VIDIOC_G_ENC_INDEX
-348565 Fix redundant condition
+348748 Fix redundant condition
n-i-bz Provide implementations of certain compiler builtins to support
compilers who may not provide those
n-i-bz Old STABS code is still being compiled, but never used. Remove it.
Modified: trunk/configure.ac
==============================================================================
--- trunk/configure.ac (original)
+++ trunk/configure.ac Fri Jun 5 22:19:06 2015
@@ -1706,7 +1706,7 @@
AC_DEFUN([AC_GCC_WARNING_SUBST_NO],[
AC_MSG_CHECKING([if gcc accepts -W$1])
safe_CFLAGS=$CFLAGS
- CFLAGS="-W$1"
+ CFLAGS="-W$1 -Werror"
AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[]], [[;]])], [
AC_SUBST([$2], [-Wno-$1])
AC_MSG_RESULT([yes])], [
@@ -1720,7 +1720,7 @@
AC_DEFUN([AC_GCC_WARNING_SUBST],[
AC_MSG_CHECKING([if gcc accepts -W$1])
safe_CFLAGS=$CFLAGS
- CFLAGS="-W$1"
+ CFLAGS="-W$1 -Werror"
AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[]], [[;]])], [
AC_SUBST([$2], [-W$1])
AC_MSG_RESULT([yes])], [
|