* [PATCH] insane: Add SH4 musl mapping to the QA arch tests
@ 2016-12-06 15:17 Vladimir Zapolskiy
2016-12-09 1:29 ` Khem Raj
0 siblings, 1 reply; 4+ messages in thread
From: Vladimir Zapolskiy @ 2016-12-06 15:17 UTC (permalink / raw)
To: openembedded-core
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 <vz@mleia.com>
---
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
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] insane: Add SH4 musl mapping to the QA arch tests
2016-12-06 15:17 [PATCH] insane: Add SH4 musl mapping to the QA arch tests Vladimir Zapolskiy
@ 2016-12-09 1:29 ` Khem Raj
2016-12-09 2:42 ` Vladimir Zapolskiy
0 siblings, 1 reply; 4+ messages in thread
From: Khem Raj @ 2016-12-09 1:29 UTC (permalink / raw)
To: Vladimir Zapolskiy; +Cc: openembedded-core
> On Dec 6, 2016, at 7:17 AM, Vladimir Zapolskiy <vz@mleia.com> 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 <vz@mleia.com>
> ---
> 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
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] insane: Add SH4 musl mapping to the QA arch tests
2016-12-09 1:29 ` Khem Raj
@ 2016-12-09 2:42 ` Vladimir Zapolskiy
2016-12-09 2:49 ` Khem Raj
0 siblings, 1 reply; 4+ messages in thread
From: Vladimir Zapolskiy @ 2016-12-09 2:42 UTC (permalink / raw)
To: Khem Raj; +Cc: openembedded-core
Hello Khem,
On 12/09/2016 03:29 AM, Khem Raj wrote:
>
>> On Dec 6, 2016, at 7:17 AM, Vladimir Zapolskiy <vz@mleia.com> 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 <vz@mleia.com>
>> ---
>> 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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] insane: Add SH4 musl mapping to the QA arch tests
2016-12-09 2:42 ` Vladimir Zapolskiy
@ 2016-12-09 2:49 ` Khem Raj
0 siblings, 0 replies; 4+ messages in thread
From: Khem Raj @ 2016-12-09 2:49 UTC (permalink / raw)
To: Vladimir Zapolskiy; +Cc: Patches and discussions about the oe-core layer
On Thu, Dec 8, 2016 at 6:42 PM, Vladimir Zapolskiy <vz@mleia.com> wrote:
> Hello Khem,
>
> On 12/09/2016 03:29 AM, Khem Raj wrote:
>>
>>> On Dec 6, 2016, at 7:17 AM, Vladimir Zapolskiy <vz@mleia.com> 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 <vz@mleia.com>
>>> ---
>>> 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
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-12-09 2:49 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-06 15:17 [PATCH] insane: Add SH4 musl mapping to the QA arch tests Vladimir Zapolskiy
2016-12-09 1:29 ` Khem Raj
2016-12-09 2:42 ` Vladimir Zapolskiy
2016-12-09 2:49 ` Khem Raj
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox