diff mbox series

[oe,meta-gnome,1/6] network-manager-applet: Package gir-1.0 files into devel package

Message ID 20180520223517.9869-1-raj.khem@gmail.com
State Superseded
Headers show
Series [oe,meta-gnome,1/6] network-manager-applet: Package gir-1.0 files into devel package | expand

Commit Message

Khem Raj May 20, 2018, 10:35 p.m. UTC
Fixes QA errors e.g.

ERROR: network-manager-applet-1.8.10-r0 do_package: QA Issue: network-manager-applet: Files/directories were installed but not shipped in any package:
  /usr/share/gir-1.0

Signed-off-by: Khem Raj <raj.khem@gmail.com>

---
 .../network-manager-applet/network-manager-applet_1.8.10.bb   | 4 ++++
 1 file changed, 4 insertions(+)

-- 
2.17.0

-- 
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Comments

Andreas Müller May 20, 2018, 10:46 p.m. UTC | #1
On Mon, May 21, 2018 at 12:35 AM, Khem Raj <raj.khem@gmail.com> wrote:
> Fixes QA errors e.g.

>

> ERROR: network-manager-applet-1.8.10-r0 do_package: QA Issue: network-manager-applet: Files/directories were installed but not shipped in any package:

>   /usr/share/gir-1.0

>

> Signed-off-by: Khem Raj <raj.khem@gmail.com>

> ---

>  .../network-manager-applet/network-manager-applet_1.8.10.bb   | 4 ++++

>  1 file changed, 4 insertions(+)

>

> diff --git a/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb b/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb

> index dc2b964be5..ccfe38d010 100644

> --- a/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb

> +++ b/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb

> @@ -43,3 +43,7 @@ FILES_${PN} += " \

>      ${datadir}/libnm-gtk/wifi.ui \

>      ${datadir}/libnma/wifi.ui \

>  "

> +

> +FILES_${PN}-dev += " \

> +    ${datadir}/gir-1.0 \

> +"

> --

> 2.17.0

>

On sumo I do not see this QA error and gir-files are packed properly
to network-manager-applet-dev.

Sure that this is a correct fix? What changed in oe-core?

Andreas
-- 
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
Khem Raj May 20, 2018, 10:47 p.m. UTC | #2
On Sun, May 20, 2018 at 3:46 PM, Andreas Müller <schnitzeltony@gmail.com> wrote:
> On Mon, May 21, 2018 at 12:35 AM, Khem Raj <raj.khem@gmail.com> wrote:
>> Fixes QA errors e.g.
>>
>> ERROR: network-manager-applet-1.8.10-r0 do_package: QA Issue: network-manager-applet: Files/directories were installed but not shipped in any package:
>>   /usr/share/gir-1.0
>>
>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> ---
>>  .../network-manager-applet/network-manager-applet_1.8.10.bb   | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb b/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
>> index dc2b964be5..ccfe38d010 100644
>> --- a/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
>> +++ b/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
>> @@ -43,3 +43,7 @@ FILES_${PN} += " \
>>      ${datadir}/libnm-gtk/wifi.ui \
>>      ${datadir}/libnma/wifi.ui \
>>  "
>> +
>> +FILES_${PN}-dev += " \
>> +    ${datadir}/gir-1.0 \
>> +"
>> --
>> 2.17.0
>>
> On sumo I do not see this QA error and gir-files are packed properly
> to network-manager-applet-dev.
>
> Sure that this is a correct fix? What changed in oe-core?
>

This is seen with musl builds. In anycase gir files if any should go
into -dev pkgs.

> Andreas
Andreas Müller May 20, 2018, 10:59 p.m. UTC | #3
On Mon, May 21, 2018 at 12:47 AM, Khem Raj <raj.khem@gmail.com> wrote:
> On Sun, May 20, 2018 at 3:46 PM, Andreas Müller <schnitzeltony@gmail.com> wrote:
>> On Mon, May 21, 2018 at 12:35 AM, Khem Raj <raj.khem@gmail.com> wrote:
>>> Fixes QA errors e.g.
>>>
>>> ERROR: network-manager-applet-1.8.10-r0 do_package: QA Issue: network-manager-applet: Files/directories were installed but not shipped in any package:
>>>   /usr/share/gir-1.0
>>>
>>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>>> ---
>>>  .../network-manager-applet/network-manager-applet_1.8.10.bb   | 4 ++++
>>>  1 file changed, 4 insertions(+)
>>>
>>> diff --git a/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb b/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
>>> index dc2b964be5..ccfe38d010 100644
>>> --- a/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
>>> +++ b/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
>>> @@ -43,3 +43,7 @@ FILES_${PN} += " \
>>>      ${datadir}/libnm-gtk/wifi.ui \
>>>      ${datadir}/libnma/wifi.ui \
>>>  "
>>> +
>>> +FILES_${PN}-dev += " \
>>> +    ${datadir}/gir-1.0 \
>>> +"
>>> --
>>> 2.17.0
>>>
>> On sumo I do not see this QA error and gir-files are packed properly
>> to network-manager-applet-dev.
>>
>> Sure that this is a correct fix? What changed in oe-core?
>>
>
> This is seen with musl builds. In anycase gir files if any should go
> into -dev pkgs.
>
Ahh musl / gir...

So how about dropping a note near FILES_${PN}-dev - otherwise chances
are high it gets removed sooner or later. For non musl this is not
necessary and at least I will not remember during next release
panic...

Andreas
diff mbox series

Patch

diff --git a/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb b/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
index dc2b964be5..ccfe38d010 100644
--- a/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
+++ b/meta-gnome/recipes-connectivity/network-manager-applet/network-manager-applet_1.8.10.bb
@@ -43,3 +43,7 @@  FILES_${PN} += " \
     ${datadir}/libnm-gtk/wifi.ui \
     ${datadir}/libnma/wifi.ui \
 "
+
+FILES_${PN}-dev += " \
+    ${datadir}/gir-1.0 \
+"