All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config
@ 2021-03-09 11:55 Anatol Belski
  2021-03-09 13:00 ` [yocto] " Joe MacDonald
  0 siblings, 1 reply; 6+ messages in thread
From: Anatol Belski @ 2021-03-09 11:55 UTC (permalink / raw)
  To: yocto; +Cc: yi.zhao

This fixes the error below:

gcc: error: unrecognized command line option
‘-fmacro-prefix-map=/path/to/build/libselinux-python/3.0-r0=/usr/src/debug/libselinux-python/3.0-r0’

Without inheriting the config, supposedly a wrong compiler is used.

Signed-off-by: Anatol Belski <anbelski@linux.microsoft.com>
---
 recipes-security/selinux/libselinux-python_3.0.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/recipes-security/selinux/libselinux-python_3.0.bb b/recipes-security/selinux/libselinux-python_3.0.bb
index 2b5438d..3c03df1 100644
--- a/recipes-security/selinux/libselinux-python_3.0.bb
+++ b/recipes-security/selinux/libselinux-python_3.0.bb
@@ -4,6 +4,8 @@ SRC_URI = "https://github.com/SELinuxProject/selinux/releases/download/${SELINUX
 
 require ${BPN}.inc
 
+inherit python3targetconfig
+
 LIC_FILES_CHKSUM = "file://LICENSE;md5=84b4d2c6ef954a2d4081e775a270d0d0"
 
 SRC_URI[md5sum] = "b387a66f087b6d97713570e85ec89d89"
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config
  2021-03-09 11:55 [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config Anatol Belski
@ 2021-03-09 13:00 ` Joe MacDonald
  2021-03-09 13:53   ` Anatol Belski
  0 siblings, 1 reply; 6+ messages in thread
From: Joe MacDonald @ 2021-03-09 13:00 UTC (permalink / raw)
  To: Anatol Belski; +Cc: yocto, yi.zhao

[-- Attachment #1: Type: text/plain, Size: 1628 bytes --]

Hi Anatol,

I will confirm this against the other branches (dunfell and gatesgarth)
but your patch doesn't appear to be against the current master branch (and
it's unlikely one patch will apply to all branches).  Can you send out
specific patches against the head of tree for each branch you care about,
please?  Then I can get them merged for you.

Thanks,
-Joe.

[[yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config] On 21.03.09 (Tue 11:55) Anatol Belski wrote:

> This fixes the error below:
> 
> gcc: error: unrecognized command line option
> ‘-fmacro-prefix-map=/path/to/build/libselinux-python/3.0-r0=/usr/src/debug/libselinux-python/3.0-r0’
> 
> Without inheriting the config, supposedly a wrong compiler is used.
> 
> Signed-off-by: Anatol Belski <anbelski@linux.microsoft.com>
> ---
>  recipes-security/selinux/libselinux-python_3.0.bb | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/recipes-security/selinux/libselinux-python_3.0.bb b/recipes-security/selinux/libselinux-python_3.0.bb
> index 2b5438d..3c03df1 100644
> --- a/recipes-security/selinux/libselinux-python_3.0.bb
> +++ b/recipes-security/selinux/libselinux-python_3.0.bb
> @@ -4,6 +4,8 @@ SRC_URI = "https://github.com/SELinuxProject/selinux/releases/download/${SELINUX
>
>  require ${BPN}.inc
>
> +inherit python3targetconfig
> +
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=84b4d2c6ef954a2d4081e775a270d0d0"
>
>  SRC_URI[md5sum] = "b387a66f087b6d97713570e85ec89d89"
> -- 
> 2.17.1
> 

> 
> 
> 


-- 
-Joe MacDonald.
:wq

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 201 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config
  2021-03-09 13:00 ` [yocto] " Joe MacDonald
@ 2021-03-09 13:53   ` Anatol Belski
  2021-03-09 15:32     ` Joe MacDonald
  0 siblings, 1 reply; 6+ messages in thread
From: Anatol Belski @ 2021-03-09 13:53 UTC (permalink / raw)
  To: Joe MacDonald; +Cc: yocto, yi.zhao

Hi Joe,

thanks for the quick check. The patch applies fine to dunfell and 
gatesgarth, master has the changed recipe version in the bb name. I was 
about to rebase to master but pulling shows already did it. I was too 
slow :) thanks for the quick fix.

Regards

Anatol

On 3/9/2021 2:00 PM, Joe MacDonald wrote:
> Hi Anatol,
>
> I will confirm this against the other branches (dunfell and gatesgarth)
> but your patch doesn't appear to be against the current master branch (and
> it's unlikely one patch will apply to all branches).  Can you send out
> specific patches against the head of tree for each branch you care about,
> please?  Then I can get them merged for you.
>
> Thanks,
> -Joe.
>
> [[yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config] On 21.03.09 (Tue 11:55) Anatol Belski wrote:
>
>> This fixes the error below:
>>
>> gcc: error: unrecognized command line option
>> ‘-fmacro-prefix-map=/path/to/build/libselinux-python/3.0-r0=/usr/src/debug/libselinux-python/3.0-r0’
>>
>> Without inheriting the config, supposedly a wrong compiler is used.
>>
>> Signed-off-by: Anatol Belski <anbelski@linux.microsoft.com>
>> ---
>>   recipes-security/selinux/libselinux-python_3.0.bb | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/recipes-security/selinux/libselinux-python_3.0.bb b/recipes-security/selinux/libselinux-python_3.0.bb
>> index 2b5438d..3c03df1 100644
>> --- a/recipes-security/selinux/libselinux-python_3.0.bb
>> +++ b/recipes-security/selinux/libselinux-python_3.0.bb
>> @@ -4,6 +4,8 @@ SRC_URI = "https://github.com/SELinuxProject/selinux/releases/download/${SELINUX
>>   
>>   require ${BPN}.inc
>>   
>> +inherit python3targetconfig
>> +
>>   LIC_FILES_CHKSUM = "file://LICENSE;md5=84b4d2c6ef954a2d4081e775a270d0d0"
>>   
>>   SRC_URI[md5sum] = "b387a66f087b6d97713570e85ec89d89"
>> -- 
>> 2.17.1
>>
>> 
>>
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config
  2021-03-09 13:53   ` Anatol Belski
@ 2021-03-09 15:32     ` Joe MacDonald
  2021-03-10  5:30       ` Yi Zhao
  0 siblings, 1 reply; 6+ messages in thread
From: Joe MacDonald @ 2021-03-09 15:32 UTC (permalink / raw)
  To: Anatol Belski; +Cc: yocto, yi.zhao

[-- Attachment #1: Type: text/plain, Size: 2563 bytes --]

[Re: [yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config] On 21.03.09 (Tue 14:53) Anatol Belski wrote:

> Hi Joe,
> 
> thanks for the quick check. The patch applies fine to dunfell and
> gatesgarth, master has the changed recipe version in the bb name. I was
> about to rebase to master but pulling shows already did it. I was too slow
> :) thanks for the quick fix.

No problem!  I assumed you were working on a just-slightly-out-of-date
master branch and my work tree already had your patch half applied when
the 'git am' failed, so I just finished that part up.

I'll get to the other branches in a bit.

-Joe.

> 
> Regards
> 
> Anatol
> 
> On 3/9/2021 2:00 PM, Joe MacDonald wrote:
> > Hi Anatol,
> > 
> > I will confirm this against the other branches (dunfell and gatesgarth)
> > but your patch doesn't appear to be against the current master branch (and
> > it's unlikely one patch will apply to all branches).  Can you send out
> > specific patches against the head of tree for each branch you care about,
> > please?  Then I can get them merged for you.
> > 
> > Thanks,
> > -Joe.
> > 
> > [[yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config] On 21.03.09 (Tue 11:55) Anatol Belski wrote:
> > 
> > > This fixes the error below:
> > > 
> > > gcc: error: unrecognized command line option
> > > ‘-fmacro-prefix-map=/path/to/build/libselinux-python/3.0-r0=/usr/src/debug/libselinux-python/3.0-r0’
> > > 
> > > Without inheriting the config, supposedly a wrong compiler is used.
> > > 
> > > Signed-off-by: Anatol Belski <anbelski@linux.microsoft.com>
> > > ---
> > >   recipes-security/selinux/libselinux-python_3.0.bb | 2 ++
> > >   1 file changed, 2 insertions(+)
> > > 
> > > diff --git a/recipes-security/selinux/libselinux-python_3.0.bb b/recipes-security/selinux/libselinux-python_3.0.bb
> > > index 2b5438d..3c03df1 100644
> > > --- a/recipes-security/selinux/libselinux-python_3.0.bb
> > > +++ b/recipes-security/selinux/libselinux-python_3.0.bb
> > > @@ -4,6 +4,8 @@ SRC_URI = "https://github.com/SELinuxProject/selinux/releases/download/${SELINUX
> > >   require ${BPN}.inc
> > > +inherit python3targetconfig
> > > +
> > >   LIC_FILES_CHKSUM = "file://LICENSE;md5=84b4d2c6ef954a2d4081e775a270d0d0"
> > >   SRC_URI[md5sum] = "b387a66f087b6d97713570e85ec89d89"
> > > -- 
> > > 2.17.1
> > > 
> > > 
> > > 
> > 

-- 
-Joe MacDonald.
:wq

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 201 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config
  2021-03-09 15:32     ` Joe MacDonald
@ 2021-03-10  5:30       ` Yi Zhao
  2021-03-10 11:14         ` Anatol Belski
  0 siblings, 1 reply; 6+ messages in thread
From: Yi Zhao @ 2021-03-10  5:30 UTC (permalink / raw)
  To: Joe MacDonald, Anatol Belski; +Cc: yocto


On 3/9/21 11:32 PM, Joe MacDonald wrote:
> [Re: [yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config] On 21.03.09 (Tue 14:53) Anatol Belski wrote:
>
>> Hi Joe,
>>
>> thanks for the quick check. The patch applies fine to dunfell and
>> gatesgarth, master has the changed recipe version in the bb name. I was
>> about to rebase to master but pulling shows already did it. I was too slow
>> :) thanks for the quick fix.
> No problem!  I assumed you were working on a just-slightly-out-of-date
> master branch and my work tree already had your patch half applied when
> the 'git am' failed, so I just finished that part up.


Hi Joe,


This patch doesn't need to be merged into master because I have fixed it 
in commit fb15056ff44318d7886fd0f68e2f6dba716e9be4.


//Yi

>
> I'll get to the other branches in a bit.
>
> -Joe.
>
>> Regards
>>
>> Anatol
>>
>> On 3/9/2021 2:00 PM, Joe MacDonald wrote:
>>> Hi Anatol,
>>>
>>> I will confirm this against the other branches (dunfell and gatesgarth)
>>> but your patch doesn't appear to be against the current master branch (and
>>> it's unlikely one patch will apply to all branches).  Can you send out
>>> specific patches against the head of tree for each branch you care about,
>>> please?  Then I can get them merged for you.
>>>
>>> Thanks,
>>> -Joe.
>>>
>>> [[yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config] On 21.03.09 (Tue 11:55) Anatol Belski wrote:
>>>
>>>> This fixes the error below:
>>>>
>>>> gcc: error: unrecognized command line option
>>>> ‘-fmacro-prefix-map=/path/to/build/libselinux-python/3.0-r0=/usr/src/debug/libselinux-python/3.0-r0’
>>>>
>>>> Without inheriting the config, supposedly a wrong compiler is used.
>>>>
>>>> Signed-off-by: Anatol Belski <anbelski@linux.microsoft.com>
>>>> ---
>>>>    recipes-security/selinux/libselinux-python_3.0.bb | 2 ++
>>>>    1 file changed, 2 insertions(+)
>>>>
>>>> diff --git a/recipes-security/selinux/libselinux-python_3.0.bb b/recipes-security/selinux/libselinux-python_3.0.bb
>>>> index 2b5438d..3c03df1 100644
>>>> --- a/recipes-security/selinux/libselinux-python_3.0.bb
>>>> +++ b/recipes-security/selinux/libselinux-python_3.0.bb
>>>> @@ -4,6 +4,8 @@ SRC_URI = "https://github.com/SELinuxProject/selinux/releases/download/${SELINUX
>>>>    require ${BPN}.inc
>>>> +inherit python3targetconfig
>>>> +
>>>>    LIC_FILES_CHKSUM = "file://LICENSE;md5=84b4d2c6ef954a2d4081e775a270d0d0"
>>>>    SRC_URI[md5sum] = "b387a66f087b6d97713570e85ec89d89"
>>>> -- 
>>>> 2.17.1
>>>>
>>>> 
>>>>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config
  2021-03-10  5:30       ` Yi Zhao
@ 2021-03-10 11:14         ` Anatol Belski
  0 siblings, 0 replies; 6+ messages in thread
From: Anatol Belski @ 2021-03-10 11:14 UTC (permalink / raw)
  To: Yi Zhao, Joe MacDonald; +Cc: yocto

[-- Attachment #1: Type: text/plain, Size: 3054 bytes --]

Hi,

On 3/10/2021 6:30 AM, Yi Zhao wrote:
>
> On 3/9/21 11:32 PM, Joe MacDonald wrote:
>> [Re: [yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] 
>> libselinux-python: Fix build error due to missing target config] On 
>> 21.03.09 (Tue 14:53) Anatol Belski wrote:
>>
>>> Hi Joe,
>>>
>>> thanks for the quick check. The patch applies fine to dunfell and
>>> gatesgarth, master has the changed recipe version in the bb name. I was
>>> about to rebase to master but pulling shows already did it. I was 
>>> too slow
>>> :) thanks for the quick fix.
>> No problem!  I assumed you were working on a just-slightly-out-of-date
>> master branch and my work tree already had your patch half applied when
>> the 'git am' failed, so I just finished that part up.
>
>
> Hi Joe,
>
>
> This patch doesn't need to be merged into master because I have fixed 
> it in commit fb15056ff44318d7886fd0f68e2f6dba716e9be4.

either way, a backport to dunfell and gatesgarth would be great to have 
then.

Thanks!

Anatol


> //Yi
>
>>
>> I'll get to the other branches in a bit.
>>
>> -Joe.
>>
>>> Regards
>>>
>>> Anatol
>>>
>>> On 3/9/2021 2:00 PM, Joe MacDonald wrote:
>>>> Hi Anatol,
>>>>
>>>> I will confirm this against the other branches (dunfell and 
>>>> gatesgarth)
>>>> but your patch doesn't appear to be against the current master 
>>>> branch (and
>>>> it's unlikely one patch will apply to all branches).  Can you send out
>>>> specific patches against the head of tree for each branch you care 
>>>> about,
>>>> please?  Then I can get them merged for you.
>>>>
>>>> Thanks,
>>>> -Joe.
>>>>
>>>> [[yocto] [meta-selinux][dunfell][gatesgarth][master][PATCH] 
>>>> libselinux-python: Fix build error due to missing target config] On 
>>>> 21.03.09 (Tue 11:55) Anatol Belski wrote:
>>>>
>>>>> This fixes the error below:
>>>>>
>>>>> gcc: error: unrecognized command line option
>>>>> ‘-fmacro-prefix-map=/path/to/build/libselinux-python/3.0-r0=/usr/src/debug/libselinux-python/3.0-r0’ 
>>>>>
>>>>>
>>>>> Without inheriting the config, supposedly a wrong compiler is used.
>>>>>
>>>>> Signed-off-by: Anatol Belski <anbelski@linux.microsoft.com>
>>>>> ---
>>>>>    recipes-security/selinux/libselinux-python_3.0.bb | 2 ++
>>>>>    1 file changed, 2 insertions(+)
>>>>>
>>>>> diff --git a/recipes-security/selinux/libselinux-python_3.0.bb 
>>>>> b/recipes-security/selinux/libselinux-python_3.0.bb
>>>>> index 2b5438d..3c03df1 100644
>>>>> --- a/recipes-security/selinux/libselinux-python_3.0.bb
>>>>> +++ b/recipes-security/selinux/libselinux-python_3.0.bb
>>>>> @@ -4,6 +4,8 @@ SRC_URI = 
>>>>> "https://github.com/SELinuxProject/selinux/releases/download/${SELINUX 
>>>>>
>>>>>    require ${BPN}.inc
>>>>> +inherit python3targetconfig
>>>>> +
>>>>>    LIC_FILES_CHKSUM = 
>>>>> "file://LICENSE;md5=84b4d2c6ef954a2d4081e775a270d0d0"
>>>>>    SRC_URI[md5sum] = "b387a66f087b6d97713570e85ec89d89"
>>>>> -- 
>>>>> 2.17.1
>>>>>
>>>>>
>>>>>
>
> 
>

[-- Attachment #2: Type: text/html, Size: 5777 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2021-03-10 11:14 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-03-09 11:55 [meta-selinux][dunfell][gatesgarth][master][PATCH] libselinux-python: Fix build error due to missing target config Anatol Belski
2021-03-09 13:00 ` [yocto] " Joe MacDonald
2021-03-09 13:53   ` Anatol Belski
2021-03-09 15:32     ` Joe MacDonald
2021-03-10  5:30       ` Yi Zhao
2021-03-10 11:14         ` Anatol Belski

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.