ruby-changes:21929
From: nobu <ko1@a...>
Date: Thu, 8 Dec 2011 12:11:25 +0900 (JST)
Subject: [ruby-changes:21929] nobu:r33978 (trunk): * configure.in (RUBY_WERROR_FLAG): append all warning flags which
nobu 2011-12-08 12:11:11 +0900 (Thu, 08 Dec 2011) New Revision: 33978 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=33978 Log: * configure.in (RUBY_WERROR_FLAG): append all warning flags which are enabled to compile, so that printf format modifiers properly fail. [ruby-core:41351] [Bug #5679] Modified files: trunk/ChangeLog trunk/configure.in Index: configure.in =================================================================== --- configure.in (revision 33977) +++ configure.in (revision 33978) @@ -418,6 +418,8 @@ dnl compiler section { AC_DEFUN([RUBY_WERROR_FLAG], [dnl +save_CFLAGS="$CFLAGS" +CFLAGS="$CFLAGS $rb_cv_warnflags" if test "${ac_c_werror_flag+set}"; then rb_c_werror_flag="$ac_c_werror_flag" else @@ -425,6 +427,8 @@ fi ac_c_werror_flag=yes $1 +CFLAGS="$save_CFLAGS" +save_CFLAGS= if test "${rb_c_werror_flag+set}"; then ac_c_werror_flag="$rb_c_werror_flag" else @@ -432,18 +436,15 @@ fi]) AC_DEFUN(RUBY_TRY_CFLAGS, [ - save_CFLAGS="$CFLAGS" - CFLAGS="[$]CFLAGS $1" AC_MSG_CHECKING([whether ]$1[ is accepted as CFLAGS]) RUBY_WERROR_FLAG([ + CFLAGS="[$]CFLAGS $1" AC_TRY_COMPILE([$4], [$5], [$2 AC_MSG_RESULT(yes)], [$3 AC_MSG_RESULT(no)]) ]) - CFLAGS="$save_CFLAGS" - save_CFLAGS= ]) AC_DEFUN(RUBY_TRY_LDFLAGS, [ Index: ChangeLog =================================================================== --- ChangeLog (revision 33977) +++ ChangeLog (revision 33978) @@ -1,3 +1,9 @@ +Thu Dec 8 12:11:06 2011 Nobuyoshi Nakada <nobu@r...> + + * configure.in (RUBY_WERROR_FLAG): append all warning flags which + are enabled to compile, so that printf format modifiers properly + fail. [ruby-core:41351] [Bug #5679] + Thu Dec 8 07:20:15 2011 Eric Hodel <drbrain@s...> * doc/re.rdoc: Document difference between match and =~, options with -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/