[OE-core] [PATCH 1/2] libx11.inc: disable warnings that don't work on certain compilers

[McClintock Matthew-B29882]

On Mon, Sep 24, 2012 at 3:00 PM, Martin Jansa <martin.jansa at gmail.com> wrote:
> On Mon, Sep 24, 2012 at 02:55:45PM -0500, Matthew McClintock wrote:
>> Fixes these sorts of issues present on older gcc (CentOS 5.x in this case)
>>
>> | cc1: error: unrecognized command line option "-Werror=implicit"
>> | cc1: error: unrecognized command line option "-Werror=nonnull"
>> | cc1: error: unrecognized command line option "-Werror=init-self"
>> | cc1: error: unrecognized command line option "-Werror=main"
>> | cc1: error: unrecognized command line option "-Werror=missing-braces"
>> | cc1: error: unrecognized command line option "-Werror=sequence-point"
>> | cc1: error: unrecognized command line option "-Werror=return-type"
>> | cc1: error: unrecognized command line option "-Werror=trigraphs"
>> | cc1: error: unrecognized command line option "-Werror=array-bounds"
>> | cc1: error: unrecognized command line option "-Werror=write-strings"
>> | cc1: error: unrecognized command line option "-Werror=address"
>> | cc1: error: unrecognized command line option "-Werror=int-to-pointer-cast"
>> | cc1: error: unrecognized command line option "-Werror=pointer-to-int-cast"
>
> Shouldn't it be applied only for -native? version?

That's reasonable. But, suppressing warnings to compile logs also did
not seem to matter much since we don't go line by line on warnings in
compile logs (does anyone?). I'll go with the consensus though.

-M

>
> Cheers,
>
>>
>> Signed-off-by: Matthew McClintock <msm at freescale.com>
>> ---
>>  meta/recipes-graphics/xorg-lib/libx11.inc |    3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-graphics/xorg-lib/libx11.inc b/meta/recipes-graphics/xorg-lib/libx11.inc
>> index 3ecd9e5..0e99442 100644
>> --- a/meta/recipes-graphics/xorg-lib/libx11.inc
>> +++ b/meta/recipes-graphics/xorg-lib/libx11.inc
>> @@ -11,7 +11,7 @@ inherit siteinfo
>>  FILESPATH = "${FILE_DIRNAME}/libx11"
>>
>>  PE = "1"
>> -INC_PR = "r8"
>> +INC_PR = "r9"
>>
>>  PROVIDES = "virtual/libx11"
>>
>> @@ -23,6 +23,7 @@ DEPENDS += "xproto xextproto xtrans libxcb kbproto inputproto"
>>  DEPENDS += "xproto-native"
>>
>>  EXTRA_OECONF += "--with-keysymdefdir=${STAGING_INCDIR}/X11/"
>> +EXTRA_OEMAKE = '-e CWARNFLAGS=""'
>>
>>  # Let people with incredibly archaic requirements enable Xcms and BigFont, but
>>  # disable them by default.
>> --
>> 1.7.9.7
>>
>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core at lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>