{"thread":{"id":"29653","subject":"[PATCH] Add --with-gcc-warnings configure option","startedAt":"2012-02-17T14:25:19Z","lastAt":"2012-02-17T14:25:19Z","messageCount":1,"participants":["Elia Pinto"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"184884","messageId":"1329488719-6991-1-git-send-email-gitter.spiros@gmail.com","threadId":"29653","inReplyTo":null,"subject":"[PATCH] Add --with-gcc-warnings configure option","fromName":"Elia Pinto","fromEmail":"gitter.spiros@gmail.com","sentAt":"2012-02-17T14:25:19Z","receivedAt":"2012-02-17T14:25:19Z","isPatch":true,"sender":{"key":"gitter.spiros@gmail.com","avatar":"https://avatars.githubusercontent.com/u/158490?v=4"},"body":"Introduce a new --with-gcc-warnings configure option\nusing a new autoconf macro that check if the compiler\nknow the option passed or not in a portable way.\n\nSigned-off-by: Elia Pinto <gitter.spiros@gmail.com>\n---\n Makefile      |    2 +-\n config.mak.in |    1 +\n configure.ac  |  118 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++\n 3 files changed, 120 insertions(+), 1 deletions(-)\n\ndiff --git a/Makefile b/Makefile\nindex a0de4e9..21bc8d4 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -310,7 +310,7 @@ endif\n \n CFLAGS = -g -O2 -Wall\n LDFLAGS =\n-ALL_CFLAGS = $(CPPFLAGS) $(CFLAGS)\n+ALL_CFLAGS = $(CPPFLAGS) $(CFLAGS) $(AM_CFLAGS)\n ALL_LDFLAGS = $(LDFLAGS)\n STRIP ?= strip\n \ndiff --git a/config.mak.in b/config.mak.in\nindex b2ba710..5b7dbfd 100644\n--- a/config.mak.in\n+++ b/config.mak.in\n@@ -2,6 +2,7 @@\n # @configure_input@\n \n CC = @CC@\n+AM_CFLAGS = @GIT_CFLAGS@\n CFLAGS = @CFLAGS@\n CPPFLAGS = @CPPFLAGS@\n LDFLAGS = @LDFLAGS@\ndiff --git a/configure.ac b/configure.ac\nindex 24190de..92dc1a9 100644\n--- a/configure.ac\n+++ b/configure.ac\n@@ -14,6 +14,34 @@ echo \"# ${config_append}.  Generated by configure.\" > \"${config_append}\"\n \n \n ## Definitions of macros\n+# git_AS_VAR_APPEND(VAR, VALUE)\n+# ----------------------------\n+# Provide the functionality of AS_VAR_APPEND if Autoconf does not have it.\n+m4_ifdef([AS_VAR_APPEND],\n+[m4_copy([AS_VAR_APPEND], [git_AS_VAR_APPEND])],\n+[m4_define([git_AS_VAR_APPEND],\n+[AS_VAR_SET([$1], [AS_VAR_GET([$1])$2])])])\n+\n+# GIT_CFLAGS_ADD(PARAMETER, [VARIABLE = GIT_CFLAGS])\n+# ------------------------------------------------\n+# Adds parameter to GIT_CFLAGS if the compiler supports it.  For example,\n+# GIT_CFLAGS_ADD([-Wall],[GIT_CFLAGS]).\n+AC_DEFUN([GIT_CFLAGS_ADD],\n+[AS_VAR_PUSHDEF([git_my_cflags], [git_cv_warn_$1])dnl\n+AC_CACHE_CHECK([whether compiler handles $1], [git_my_cflags], [\n+  save_CFLAGS=\"$CFLAGS\"\n+  CFLAGS=\"${CFLAGS} $1\"\n+  AC_COMPILE_IFELSE([AC_LANG_PROGRAM([])],\n+                    [AS_VAR_SET([git_my_cflags], [yes])],\n+                    [AS_VAR_SET([git_my_cflags], [no])])\n+  CFLAGS=\"$save_CFLAGS\"\n+])\n+AS_VAR_PUSHDEF([git_cflags], m4_if([$2], [], [[GIT_CFLAGS]], [[$2]]))dnl\n+AS_VAR_IF([git_my_cflags], [yes], [git_AS_VAR_APPEND([git_cflags], [\" $1\"])])\n+AS_VAR_POPDEF([git_cflags])dnl\n+AS_VAR_POPDEF([git_my_cflags])dnl\n+m4_ifval([$2], [AS_LITERAL_IF([$2], [AC_SUBST([$2])], [])])dnl\n+])\n # GIT_CONF_APPEND_LINE(LINE)\n # --------------------------\n # Append LINE to file ${config_append}\n@@ -158,6 +186,96 @@ if test -z \"$lib\"; then\n    lib=lib\n fi\n \n+# Turn gcc warning\n+\n+AC_ARG_ENABLE([gcc-warnings],\n+  [AS_HELP_STRING([--enable-gcc-warnings],\n+                  [turn on GCC warnings (for developers)@<:@default=no@:>@])],\n+  [case $enableval in\n+     yes|no) ;;\n+     *)      AC_MSG_ERROR([bad value $enableval for gcc-warnings option]) ;;\n+   esac\n+   git_gcc_warnings=$enableval],\n+  [git_gcc_warnings=no]\n+)\n+\n+AS_IF([test \"x$git_gcc_warnings\" = xyes],\n+  [ # Add/Delete as needed\n+  MAX_STACK_SIZE=32768\n+  GIT_CFLAGS_ADD([-Wall], [GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-pedantic], [GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wextra], [GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wformat-y2k], [GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-fdiagnostics-show-option],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-funit-at-a-time],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-fstrict-aliasing],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wstrict-overflow],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-fstrict-overflow],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wpointer-arith],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wundef],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wformat-security],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Winit-self],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wmissing-include-dirs],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wunused],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wunknown-pragmas],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wstrict-aliasing],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wshadow],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wbad-function-cast],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wcast-align],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wwrite-strings],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wlogical-op],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Waggregate-return],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wstrict-prototypes],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wold-style-definition],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wmissing-prototypes],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wmissing-declarations],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wmissing-noreturn],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wmissing-format-attribute],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wredundant-decls],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wnested-externs],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Winline],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Winvalid-pch],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wvolatile-register-var],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wdisabled-optimization],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wbuiltin-macro-redefined],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wmudflap],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wpacked-bitfield-compat],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wsync-nand],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wattributes],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wcoverage-mismatch],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wmultichar],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wcpp],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wdeprecated-declarations],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wdiv-by-zero],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wdouble-promotion],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wendif-labels],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wformat-contains-nul],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wformat-extra-args],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wformat-zero-length],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wformat=2],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wmultichar],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wnormalized=nfc],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Woverflow],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wpointer-to-int-cast],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wpragmas],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wsuggest-attribute=const],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wsuggest-attribute=noreturn],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wsuggest-attribute=pure],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wtrampolines],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wno-missing-field-initializers],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wno-sign-compare],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wjump-misses-init],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wno-format-nonliteral],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wframe-larger-than=$MAX_STACK_SIZE],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-fstack-protector-all],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-fasynchronous-unwind-tables],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-fdiagnostics-show-option],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-funit-at-a-time],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-fipa-pure-const],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wno-aggregate-return],[GIT_CFLAGS])\n+  GIT_CFLAGS_ADD([-Wno-redundant-decls],[GIT_CFLAGS])\n+  AC_SUBST([GIT_CFLAGS])\n+  ])\n AC_ARG_ENABLE([pthreads],\n  [AS_HELP_STRING([--enable-pthreads=FLAGS],\n   [FLAGS is the value to pass to the compiler to enable POSIX Threads.]\n-- \n1.7.8.rc3.31.g017d1\n"}]}