[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:06 PM, Matthew McClintock <msm at freescale.com> wrote:
> 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.

Actually, this makes sense now because I'm seeing issues with
non-nativesdk packages with the '-e' I've added... thought I build
tested that... ;(

-M

>
> -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
>>