Re: [PATCH] fix build failure w. gcc-4.3.3

[Mike Frysinger <vapier@gentoo.org>]

[-- Attachment #1: Type: text/plain, Size: 1449 bytes --]

On Saturday 31 January 2009 12:13:26 Manuel Lauss wrote:
> On Sat, 31 Jan 2009 11:56:05 -0500 Mike Frysinger wrote:
> > On Saturday 31 January 2009 03:46:05 Manuel Lauss wrote:
> > > On Fri, 30 Jan 2009 11:28:54 +0100 Manuel Lauss wrote:
> > > > gcc-4.3.3 likes to have printf() and related functions to have
> > > > at least 2 arguments when the format string is not a constant:
> > > >
> > > > cc1: warnings being treated as error
> > > > arch/sh/kernel/setup.c: In function 'early_parse_mem':
> > > > arch/sh/kernel/setup.c:111: error: format not a string literal and no
> > > > format arguments
> > >
> > > Please disregard this patch:  turns out that Gentoo applied a patch
> > > which unconditionally enables -Wformat-security and produces this
> > > message.
> >
> > that's incorrect.  the patch sets format-security on by default, not
> > unconditionally.  having the common kernel itself build with -Wno-format-
> > security probably is worthwhile.
>
> Well, at least here -Wno-format-security doesn't work, neither on x86,
> sh nor mips.  Otherwise I would have sent a patch doing exactly that.

works fine for me.  not that it matters as the kernel already can handle this:
KBUILD_CFLAGS += $(call cc-option,-Wno-format-security,)

also, this isnt a Gentoo specific issue.  other distros are enabling this flag 
by default as well.  they just dont make cross-compiler generation as easy as 
Gentoo ;).
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 835 bytes --]