Thanks for the catch. I've also added -errwarn=E_NO_IMPLICIT_DECL_ALLOWED
Do you want to add corresponding SunCC flags for the other -Werror=? On Nov 1, 2011, at 1:37 PM, Alan Coopersmith wrote: > I only see one argument to AC_LANG_SOURCE in > https://www.gnu.org/software/autoconf/manual/html_node/Generating-Sources.html > > Did you mean to change the macro to AC_LANG_PROGRAM? > > -alan- > > On 11/01/11 13:16, Jeremy Huddleston wrote: >> The current test was always failing to compile because 'return 0;' was >> global rather than in the body of a function. >> >> Signed-off-by: Jeremy Huddleston<[email protected]> >> --- >> xorg-macros.m4.in | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in >> index 0527dfe..3b02190 100644 >> --- a/xorg-macros.m4.in >> +++ b/xorg-macros.m4.in >> @@ -1431,7 +1431,7 @@ if test "x$GCC" = xyes ; then >> AC_MSG_CHECKING([if $CC supports -Werror=attributes]) >> save_CFLAGS="$CFLAGS" >> CFLAGS="$CFLAGS $STRICT_CFLAGS -Werror=attributes" >> - AC_COMPILE_IFELSE([AC_LANG_SOURCE([return 0;])], >> + AC_COMPILE_IFELSE([AC_LANG_SOURCE([], [return 0;])], >> [STRICT_CFLAGS="$STRICT_CFLAGS -Werror=attributes" >> AC_MSG_RESULT([yes])], >> [AC_MSG_RESULT([no])]) > > > -- > -Alan Coopersmith- [email protected] > Oracle Solaris Platform Engineering: X Window System > _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
