[oe] [meta-oe][PATCH] luajit: mips and mipsel are 32bit targets too

[Andre McCurdy]

On Wed, Jan 25, 2017 at 10:29 AM, Andre McCurdy <armccurdy at gmail.com> wrote:
> On Wed, Jan 25, 2017 at 12:58 AM, André Draszik <git at andred.net> wrote:
>> On Tue, 2017-01-24 at 11:19 -0800, Andre McCurdy wrote:
>>> When cross compiling, the word size of the compiler set via HOST_CC
>>> must match the word size of the target. That's achieved by appending
>>> "-m32" to BUILD_CC_ARCH if the target word size is known to be 32bits.
>>>
>>> Unfortunately the current list of over-rides (powerpc, x86 and arm)
>>> does not cover all cases. Add mips and mipsel to the list too (which
>>> is still not enough to cover all targets or corner cases such as x32,
>>> but better than before).
>>>
>>> Signed-off-by: Andre McCurdy <armccurdy at gmail.com>
>>> ---
>>>  meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb b/meta-
>>> oe/recipes-devtools/luajit/luajit_2.0.4.bb
>>> index 964dc1d..2049846 100644
>>> --- a/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb
>>> +++ b/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb
>>> @@ -24,6 +24,8 @@ BBCLASSEXTEND = "native"
>>>  BUILD_CC_ARCH_append_powerpc = ' -m32'
>>>  BUILD_CC_ARCH_append_x86 = ' -m32'
>>>  BUILD_CC_ARCH_append_arm = ' -m32'
>>> +BUILD_CC_ARCH_append_mips = ' -m32'
>>> +BUILD_CC_ARCH_append_mipsel = ' -m32'
>>
>> I suspect this should be using:
>>
>> BUILD_CC_ARCH_append_mipsarchn32 = ' -m32'
>> BUILD_CC_ARCH_append_mipsarcho32 = ' -m32'
>
> That wouldn't work so well when backported to krogoth and morty (which
> is where I personally need the fix).
>
> But maybe there's a more generic way to answer the question "is the
> target 32bit?" from within a recipe without needing to keep a list of
> over-rides up to date?

SITEINFO_BITS seems to do that.

>>
>> Cheers,
>> Andre'
>>