Re: [OE-core] [PATCH] insane: Add SH4 musl mapping to the QA arch tests

2016-12-08 Thread Khem Raj 
On Thu, Dec 8, 2016 at 6:42 PM, Vladimir Zapolskiy  wrote:
> Hello Khem,
>
> On 12/09/2016 03:29 AM, Khem Raj wrote:
>>
>>> On Dec 6, 2016, at 7:17 AM, Vladimir Zapolskiy  wrote:
>>>
>>> This change allows to pass QA for packages built with sh4-oe-linux-musl
>>> toolchain, the problem is reproted while building core-image-minimal target:
>>>
>>>  ERROR: readline-7.0-r0 do_package_qa:
>>>  Error executing a python function in exec_python_func() autogenerated
>>>
>>> Signed-off-by: Vladimir Zapolskiy 
>>> ---
>>> meta/classes/insane.bbclass | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
>>> index 5ddb87b..01494e3 100644
>>> --- a/meta/classes/insane.bbclass
>>> +++ b/meta/classes/insane.bbclass
>>> @@ -138,6 +138,7 @@ def package_qa_get_machine_dict(d):
>>> "microblaze":  (189, 0,0,  False,   
>>>   32),
>>> "microblazeeb":(189, 0,0,  False,   
>>>   32),
>>> "microblazeel":(189, 0,0,  True,
>>>   32),
>>> +"sh4":(  42, 0,0,  True,   
>>>32),
>>
>> Can you also add sh4eb as well while you are at it ?
>>
>
> I don't build or do runtime testing of sh4eb, and you may notice that
> there is no any other sh4eb target in the file at the moment. I'll try
> to find some time in the future and test sh4eb linux, linux-musl and
> linux-uclibc toolchains and add all of them together as a distinct
> change.
>
> Since missing sh4 linux-musl hits me immediately, I would like to ask
> you to apply the change as is.
>

OK, I think thats fine, I was hoping that you were testing that too

> Thank you.
>
> --
> With best wishes,
> Vladimir
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] insane: Add SH4 musl mapping to the QA arch tests

2016-12-08 Thread Vladimir Zapolskiy 
Hello Khem,

On 12/09/2016 03:29 AM, Khem Raj wrote:
> 
>> On Dec 6, 2016, at 7:17 AM, Vladimir Zapolskiy  wrote:
>>
>> This change allows to pass QA for packages built with sh4-oe-linux-musl
>> toolchain, the problem is reproted while building core-image-minimal target:
>>
>>  ERROR: readline-7.0-r0 do_package_qa:
>>  Error executing a python function in exec_python_func() autogenerated
>>
>> Signed-off-by: Vladimir Zapolskiy 
>> ---
>> meta/classes/insane.bbclass | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
>> index 5ddb87b..01494e3 100644
>> --- a/meta/classes/insane.bbclass
>> +++ b/meta/classes/insane.bbclass
>> @@ -138,6 +138,7 @@ def package_qa_get_machine_dict(d):
>> "microblaze":  (189, 0,0,  False,
>>  32),
>> "microblazeeb":(189, 0,0,  False,
>>  32),
>> "microblazeel":(189, 0,0,  True, 
>>  32),
>> +"sh4":(  42, 0,0,  True,
>>   32),
> 
> Can you also add sh4eb as well while you are at it ?
> 

I don't build or do runtime testing of sh4eb, and you may notice that
there is no any other sh4eb target in the file at the moment. I'll try
to find some time in the future and test sh4eb linux, linux-musl and
linux-uclibc toolchains and add all of them together as a distinct
change.

Since missing sh4 linux-musl hits me immediately, I would like to ask
you to apply the change as is.

Thank you.

--
With best wishes,
Vladimir
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] insane: Add SH4 musl mapping to the QA arch tests

2016-12-08 Thread Khem Raj 

> On Dec 6, 2016, at 7:17 AM, Vladimir Zapolskiy  wrote:
> 
> This change allows to pass QA for packages built with sh4-oe-linux-musl
> toolchain, the problem is reproted while building core-image-minimal target:
> 
>  ERROR: readline-7.0-r0 do_package_qa:
>  Error executing a python function in exec_python_func() autogenerated
> 
> Signed-off-by: Vladimir Zapolskiy 
> ---
> meta/classes/insane.bbclass | 1 +
> 1 file changed, 1 insertion(+)
> 
> diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
> index 5ddb87b..01494e3 100644
> --- a/meta/classes/insane.bbclass
> +++ b/meta/classes/insane.bbclass
> @@ -138,6 +138,7 @@ def package_qa_get_machine_dict(d):
> "microblaze":  (189, 0,0,  False, 
> 32),
> "microblazeeb":(189, 0,0,  False, 
> 32),
> "microblazeel":(189, 0,0,  True,  
> 32),
> +"sh4":(  42, 0,0,  True, 
>  32),

Can you also add sh4eb as well while you are at it ?

>   },
> "uclinux-uclibc" : {
> "bfin":   ( 106, 0,0,  True,  
>32),
> -- 
> 2.10.2
> 
> -- 
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] insane: Add SH4 musl mapping to the QA arch tests

2016-12-06 Thread Vladimir Zapolskiy 
This change allows to pass QA for packages built with sh4-oe-linux-musl
toolchain, the problem is reproted while building core-image-minimal target:

  ERROR: readline-7.0-r0 do_package_qa:
  Error executing a python function in exec_python_func() autogenerated

Signed-off-by: Vladimir Zapolskiy 
---
 meta/classes/insane.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 5ddb87b..01494e3 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -138,6 +138,7 @@ def package_qa_get_machine_dict(d):
 "microblaze":  (189, 0,0,  False,  
   32),
 "microblazeeb":(189, 0,0,  False,  
   32),
 "microblazeel":(189, 0,0,  True,   
   32),
+"sh4":(  42, 0,0,  True,   
   32),
   },
 "uclinux-uclibc" : {
 "bfin":   ( 106, 0,0,  True,   
  32),
-- 
2.10.2

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core