On Wed, Feb 22, 2012 at 11:25:22AM -0500, Lennart Sorensen wrote: > That actually looks like a mistake. > > HOST_CFLAGS="$HOST_CFLAGS $WARN_FLAGS" > TARGET_CFLAGS="$TARGET_CFLAGS $WARN_FLAGS -g -Wredundant-decls > -Wmissing-prototypes" > > but WARN_FLAGS="-Wall -W -Wshadow -Wold-style-declaration > -Wold-style-definition -Wpointer-arith -Wundef -Wextra -Waddress > -Warray-bounds -Wattributes -Wbuiltin-macro-redefined -Wcast-align > -Wchar-subscripts -Wclobbered -Wcomment -Wcoverage-mismatch -Wdeprecated > -Wdeprecated-declarations -Wdisabled-optimization -Wdiv-by-zero -Wempty-body > -Wendif-labels -Wfloat-equal -Wformat-contains-nul -Wformat-extra-args > -Wformat-security -Wformat-y2k -Wignored-qualifiers -Wimplicit > -Wimplicit-function-declaration -Wimplicit-int -Winit-self > -Wint-to-pointer-cast -Winvalid-pch -Wunsafe-loop-optimizations -Wlogical-op > -Wmain -Wmissing-braces -Wmissing-field-initializers > -Wmissing-format-attribute -Wmissing-noreturn -Wmudflap -Wmultichar -Wnonnull > -Woverflow -Wpacked-bitfield-compat -Wparentheses -Wpointer-arith > -Wpointer-to-int-cast -Wreturn-type -Wsequence-point -Wshadow -Wsign-compare > -Wstrict-aliasing -Wswitch -Wsync-nand -Wtrigraphs -Wtype-limits -Wundef > -Wuninitialized -Wunknown-pragmas -Wunused -Wunused-function -Wun > used-label -Wunused-parameter -Wunused-result -Wunused-value > -Wunused-variable -Wvariadic-macros -Wvolatile-register-var -Wwrite-strings > -Wmissing-declarations -Wmissing-parameter-type -Wmissing-prototypes > -Wnested-externs -Wstrict-prototypes -Wpointer-sign" > > So by having -Wmissing-prototypes in WARN_FLAGS it gets set twice in > TARGET_CFLAGS and also set once in HOST_CFLAGS. Perhaps that was > not meant to be in there and should only have been in TARGET_CFLAGS.
So I removed -Wmissing-prototypes from configure.ac, and now instead I get: gcc-4.6 -DHAVE_CONFIG_H -I. -I../.. -Wall -W -I./include -DGRUB_UTIL=1 -DGRUB_LIBDIR=\"/usr/lib/grub\" -DLOCALEDIR=\"/usr/share/locale\" -DGRUB_MACHINE_EMU=1 -DGRUB_MACHINE=POWERPC_EMU -DGRUB_TARGET_CPU_POWERPC=1 -DGRUB_FILE=\"util/grub-mkimage.c\" -I. -I../.. -I. -I../.. -I../../include -I./include -I./grub-core/gnulib -I../../grub-core/gnulib -DGRUB_PKGLIBROOTDIR=\"/usr/lib/grub\" -g -Wall -O2 -Wall -W -Wshadow -Wold-style-declaration -Wold-style-definition -Wpointer-arith -Wundef -Wextra -Waddress -Warray-bounds -Wattributes -Wbuiltin-macro-redefined -Wcast-align -Wchar-subscripts -Wclobbered -Wcomment -Wcoverage-mismatch -Wdeprecated -Wdeprecated-declarations -Wdisabled-optimization -Wdiv-by-zero -Wempty-body -Wendif-labels -Wfloat-equal -Wformat-contains-nul -Wformat-extra-args -Wformat-security -Wformat-y2k -Wignored-qualifiers -Wimplicit -Wimplicit-function-declaration -Wimplicit-int -Winit-self -Wint-to-pointer-cast -Winvalid-pch -Wunsafe-loop-optimizations -Wlogical-op -Wmain -Wmissing-braces -Wmissing-field-initializers -Wmissing-format-attribute -Wmissing-noreturn -Wmudflap -Wmultichar -Wnonnull -Woverflow -Wpacked-bitfield-compat -Wparentheses -Wpointer-arith -Wpointer-to-int-cast -Wreturn-type -Wsequence-point -Wshadow -Wsign-compare -Wstrict-aliasing -Wswitch -Wsync-nand -Wtrigraphs -Wtype-limits -Wundef -Wuninitialized -Wunknown-pragmas -Wunused -Wunused-function -Wunused-label -Wunused-parameter -Wunused-result -Wunused-value -Wunused-variable -Wvariadic-macros -Wvolatile-register-var -Wwrite-strings -Wmissing-declarations -Wmissing-parameter-type -Wnested-externs -Wstrict-prototypes -Wpointer-sign -Wno-undef -Wno-sign-compare -Wno-unused -Wno-unused-parameter -Wno-redundant-decls -Wno-unreachable-code -Wno-conversion -Wno-old-style-definition -c -o util/grub_mkimage-grub-mkimage.o `test -f 'util/grub-mkimage.c' || echo '../../'`util/grub-mkimage.c In file included from ../../util/grub-mkimage.c:45:0: ../../grub-core/gnulib/argp.h:610:1: warning: no previous declaration for 'argp_usage' [-Wmissing-declarations] ../../grub-core/gnulib/argp.h:616:1: warning: no previous declaration for '_option_is_short' [-Wmissing-declarations] ../../grub-core/gnulib/argp.h:628:1: warning: no previous declaration for '_option_is_end' [-Wmissing-declarations] Does -Wmissing-declarations have to go too? -- Len Sorensen _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel