[meta-oe][PATCH] iwd: use internal ell

[meta-oe][PATCH] iwd: use internal ell

[Markus Volk]

iwd and ell need to be updated in sync. This is regularly neglected.
Also the fact that they reside in different layers compicates the update process.

Beside iwd, there are not a lot of consumers for ell.

Building with internal ell makes iwd updates easier

Signed-off-by: Markus Volk <f_l_k@t-online.de>
---
 meta-oe/recipes-connectivity/iwd/iwd_2.20.bb | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb b/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
index ffdab0260..16d900ddc 100644
--- a/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
+++ b/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
@@ -3,7 +3,7 @@ HOMEPAGE = "https://iwd.wiki.kernel.org/"
 LICENSE = "LGPL-2.1-only"
 LIC_FILES_CHKSUM = "file://COPYING;md5=fb504b67c50331fc78734fed90fb0e09"
 
-DEPENDS = "ell"
+DEPENDS = "dbus"
 
 SRC_URI = "https://www.kernel.org/pub/linux/network/wireless/${BP}.tar.xz \
            file://0001-build-Use-abs_top_srcdir-instead-of-abs_srcdir-for-e.patch \
@@ -24,8 +24,6 @@ PACKAGECONFIG[wired] = "--enable-wired,--disable-wired"
 PACKAGECONFIG[ofono] = "--enable-ofono,--disable-ofono"
 PACKAGECONFIG[systemd] = "--with-systemd-unitdir=${systemd_system_unitdir},--disable-systemd-service,systemd"
 
-EXTRA_OECONF = "--enable-external-ell"
-
 SYSTEMD_SERVICE:${PN} = " \
     iwd.service \
     ${@bb.utils.contains('PACKAGECONFIG', 'wired', 'ead.service', '', d)} \
-- 
2.46.0

Re: [oe] [meta-oe][PATCH] iwd: use internal ell

[Khem Raj]

On Sat, Sep 7, 2024 at 1:33 AM Markus Volk via lists.openembedded.org
<f_l_k=t-online.de@lists.openembedded.org> wrote:
>
> iwd and ell need to be updated in sync. This is regularly neglected.
> Also the fact that they reside in different layers compicates the update process.
>

yes this has happened in past as well.

> Beside iwd, there are not a lot of consumers for ell.
>

if this is the case then, I would like to suggest that either iwd moves to core
or ell moves out of core into meta-oe next to iwd. I think ideally it
can be deleted
but there might be other consumers of ell outside of meta-openembedded
layers and core

> Building with internal ell makes iwd updates easier
>

Building with internal ell is fine if its not generating duplicate
copies of ell artifacts. Can you confirm what happens when
switching between internal and external ell ?

> Signed-off-by: Markus Volk <f_l_k@t-online.de>
> ---
>  meta-oe/recipes-connectivity/iwd/iwd_2.20.bb | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb b/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
> index ffdab0260..16d900ddc 100644
> --- a/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
> +++ b/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
> @@ -3,7 +3,7 @@ HOMEPAGE = "https://iwd.wiki.kernel.org/"
>  LICENSE = "LGPL-2.1-only"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=fb504b67c50331fc78734fed90fb0e09"
>
> -DEPENDS = "ell"
> +DEPENDS = "dbus"
>
>  SRC_URI = "https://www.kernel.org/pub/linux/network/wireless/${BP}.tar.xz \
>             file://0001-build-Use-abs_top_srcdir-instead-of-abs_srcdir-for-e.patch \
> @@ -24,8 +24,6 @@ PACKAGECONFIG[wired] = "--enable-wired,--disable-wired"
>  PACKAGECONFIG[ofono] = "--enable-ofono,--disable-ofono"
>  PACKAGECONFIG[systemd] = "--with-systemd-unitdir=${systemd_system_unitdir},--disable-systemd-service,systemd"
>
> -EXTRA_OECONF = "--enable-external-ell"
> -
>  SYSTEMD_SERVICE:${PN} = " \
>      iwd.service \
>      ${@bb.utils.contains('PACKAGECONFIG', 'wired', 'ead.service', '', d)} \
> --
> 2.46.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#112113): https://lists.openembedded.org/g/openembedded-devel/message/112113
> Mute This Topic: https://lists.openembedded.org/mt/108319324/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>

Re: [oe] [meta-oe][PATCH] iwd: use internal ell

[Martin Jansa]

At least ofono in oe-core is using ell by default
https://git.openembedded.org/openembedded-core/tree/meta/recipes-connectivity/ofono/ofono_2.10.bb

explicitly the external on (from oe-core) since
https://git.openembedded.org/openembedded-core/commit/meta/recipes-connectivity/ofono?id=3683f524dc60325b3f3b1bc3a32e70a41064ae03

On Sat, Sep 7, 2024 at 5:24 PM Khem Raj via lists.openembedded.org
<raj.khem=gmail.com@lists.openembedded.org> wrote:
>
> On Sat, Sep 7, 2024 at 1:33 AM Markus Volk via lists.openembedded.org
> <f_l_k=t-online.de@lists.openembedded.org> wrote:
> >
> > iwd and ell need to be updated in sync. This is regularly neglected.
> > Also the fact that they reside in different layers compicates the update process.
> >
>
> yes this has happened in past as well.
>
> > Beside iwd, there are not a lot of consumers for ell.
> >
>
> if this is the case then, I would like to suggest that either iwd moves to core
> or ell moves out of core into meta-oe next to iwd. I think ideally it
> can be deleted
> but there might be other consumers of ell outside of meta-openembedded
> layers and core
>
> > Building with internal ell makes iwd updates easier
> >
>
> Building with internal ell is fine if its not generating duplicate
> copies of ell artifacts. Can you confirm what happens when
> switching between internal and external ell ?
>
> > Signed-off-by: Markus Volk <f_l_k@t-online.de>
> > ---
> >  meta-oe/recipes-connectivity/iwd/iwd_2.20.bb | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb b/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
> > index ffdab0260..16d900ddc 100644
> > --- a/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
> > +++ b/meta-oe/recipes-connectivity/iwd/iwd_2.20.bb
> > @@ -3,7 +3,7 @@ HOMEPAGE = "https://iwd.wiki.kernel.org/"
> >  LICENSE = "LGPL-2.1-only"
> >  LIC_FILES_CHKSUM = "file://COPYING;md5=fb504b67c50331fc78734fed90fb0e09"
> >
> > -DEPENDS = "ell"
> > +DEPENDS = "dbus"
> >
> >  SRC_URI = "https://www.kernel.org/pub/linux/network/wireless/${BP}.tar.xz \
> >             file://0001-build-Use-abs_top_srcdir-instead-of-abs_srcdir-for-e.patch \
> > @@ -24,8 +24,6 @@ PACKAGECONFIG[wired] = "--enable-wired,--disable-wired"
> >  PACKAGECONFIG[ofono] = "--enable-ofono,--disable-ofono"
> >  PACKAGECONFIG[systemd] = "--with-systemd-unitdir=${systemd_system_unitdir},--disable-systemd-service,systemd"
> >
> > -EXTRA_OECONF = "--enable-external-ell"
> > -
> >  SYSTEMD_SERVICE:${PN} = " \
> >      iwd.service \
> >      ${@bb.utils.contains('PACKAGECONFIG', 'wired', 'ead.service', '', d)} \
> > --
> > 2.46.0
> >
> >
> >
> >
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#112114): https://lists.openembedded.org/g/openembedded-devel/message/112114
> Mute This Topic: https://lists.openembedded.org/mt/108319324/3617156
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [martin.jansa@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>

Re: [oe] [meta-oe][PATCH] iwd: use internal ell

[Markus Volk]

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

On Sat, Sep 7 2024 at 05:44:19 PM +02:00:00, Martin Jansa 
<martin.jansa@gmail.com> wrote:
> explicitly the external on (from oe-core) since
> <https://git.openembedded.org/openembedded-core/commit/meta/recipes-connectivity/ofono?id=3683f524dc60325b3f3b1bc3a32e70a41064ae03>

Understand. I also know that bluez5 uses ell when PACKAGECONFIG is 
enabled for either mesh or btpclient. But that's not the default. So 
for consistency, maybe it would be good to keep it external for all 
three?

I'm not sure about this change either, but wanted to bring it up for 
discussion because it always feels a bit like groundhog day with 
iwd/ell.

I'm not that familiar with investigating sstate cache reuse issues. I'm 
willing to learn more about it, but right now I don't know what exact 
files I would need to compare, or is there something like a tool that 
can be used for this purpose?


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

Re: [oe] [meta-oe][PATCH] iwd: use internal ell

[Khem Raj]

On Sat, Sep 7, 2024 at 11:22 AM Markus Volk <f_l_k@t-online.de> wrote:
>
> On Sat, Sep 7 2024 at 05:44:19 PM +02:00:00, Martin Jansa <martin.jansa@gmail.com> wrote:
>
> explicitly the external on (from oe-core) since https://git.openembedded.org/openembedded-core/commit/meta/recipes-connectivity/ofono?id=3683f524dc60325b3f3b1bc3a32e70a41064ae03
>
>
> Understand. I also know that bluez5 uses ell when PACKAGECONFIG is enabled for either mesh or btpclient. But that's not the default. So for consistency, maybe it would be good to keep it external for all three?
>
> I'm not sure about this change either, but wanted to bring it up for discussion because it always feels a bit like groundhog day with iwd/ell.
>
> I'm not that familiar with investigating sstate cache reuse issues. I'm willing to learn more about it, but right now I don't know what exact files I would need to compare, or is there something like a tool that can be used for this purpose?

I think ell is in core layer therefore it binds us to the compatible
versions we can use for packages which uses ell. It also depends upon
how upstream is expecting distros to consume it. If they think that
it should be vendored in apps then thats what we should do

Re: [oe] [meta-oe][PATCH] iwd: use internal ell

[Markus Volk]

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

On Sat, Sep 7 2024 at 12:56:04 PM -07:00:00, Khem Raj 
<raj.khem@gmail.com> wrote:
> If they think that
> it should be vendored in apps then thats what we should do

By default, all three known recipes that use ell are probably vendored 
by default. The --enable-external-ell option is set in the recipe for 
all of them.


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