public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* [openembedded-core][scarthgap][PATCH] tcp-wrappers: extend recipe for native/nativesdk builds
@ 2026-03-09 14:56 Deepak Rathore -X (deeratho - E INFOCHIPS PRIVATE LIMITED at Cisco)
  2026-03-18 16:51 ` [OE-core] " Yoann Congal
  0 siblings, 1 reply; 3+ messages in thread
From: Deepak Rathore -X (deeratho - E INFOCHIPS PRIVATE LIMITED at Cisco) @ 2026-03-09 14:56 UTC (permalink / raw)
  To: openembedded-core

From: Deepak Rathore <deeratho@cisco.com>

Some SDK dependency chains require tcp-wrappers to be available
as host/SDK artifacts. The current metadata only partially provides this,
which can lead to dependency resolution failures when this recipe is pulled
into SDK-oriented builds.

This change does not alter target package behavior; it only enables required
native/nativesdk variants for build and SDK integration paths.

Signed-off-by: Deepak Rathore <deeratho@cisco.com>

diff --git a/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb b/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
index bcd1d6f792..26eaeab1ac 100644
--- a/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
+++ b/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
@@ -132,3 +132,4 @@ do_install () {
 }
 
 FILES:${PN} += "${sysconfdir}/hosts.allow ${sysconfdir}/hosts.deny"
+BBCLASSEXTEND = "native nativesdk"
-- 
2.35.6



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

* Re: [OE-core] [openembedded-core][scarthgap][PATCH] tcp-wrappers: extend recipe for native/nativesdk builds
  2026-03-09 14:56 [openembedded-core][scarthgap][PATCH] tcp-wrappers: extend recipe for native/nativesdk builds Deepak Rathore -X (deeratho - E INFOCHIPS PRIVATE LIMITED at Cisco)
@ 2026-03-18 16:51 ` Yoann Congal
  2026-03-18 16:55   ` Richard Purdie
  0 siblings, 1 reply; 3+ messages in thread
From: Yoann Congal @ 2026-03-18 16:51 UTC (permalink / raw)
  To: deeratho, openembedded-core

On Mon Mar 9, 2026 at 3:56 PM CET, Deepak Rathore via lists.openembedded.org wrote:
> From: Deepak Rathore <deeratho@cisco.com>
>
> Some SDK dependency chains require tcp-wrappers to be available
> as host/SDK artifacts. The current metadata only partially provides this,
> which can lead to dependency resolution failures when this recipe is pulled
> into SDK-oriented builds.
>
> This change does not alter target package behavior; it only enables required
> native/nativesdk variants for build and SDK integration paths.
>
> Signed-off-by: Deepak Rathore <deeratho@cisco.com>
>
> diff --git a/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb b/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
> index bcd1d6f792..26eaeab1ac 100644
> --- a/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
> +++ b/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
> @@ -132,3 +132,4 @@ do_install () {
>  }
>  
>  FILES:${PN} += "${sysconfdir}/hosts.allow ${sysconfdir}/hosts.deny"
> +BBCLASSEXTEND = "native nativesdk"

That effectively create 2 new recipes tcp-wrappers-native and
nativesdk-tcp-wrappers. That is not a trivial change that I can take
without a clear and strong justification.

What "dependency resolution failures" did you see?

Also, can you check that tcp-wrapper-native builds on Fedora 41? This is
the scarthgap supported distro with the most recent GCC : gcc-14 (and it
usually trigger build failures on old/unmaintained packages)

Finally, OE-Core master did remove the recipe for it being really
unmaintained[0]. This is not a good sign :/

[0]: https://git.openembedded.org/openembedded-core/commit/?h=master-next&id=e556df28f47e754b53e1f46c97dde2b19fd8fad1

Regards,
-- 
Yoann Congal
Smile ECS



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

* Re: [OE-core] [openembedded-core][scarthgap][PATCH] tcp-wrappers: extend recipe for native/nativesdk builds
  2026-03-18 16:51 ` [OE-core] " Yoann Congal
@ 2026-03-18 16:55   ` Richard Purdie
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Purdie @ 2026-03-18 16:55 UTC (permalink / raw)
  To: yoann.congal, deeratho, openembedded-core

On Wed, 2026-03-18 at 17:51 +0100, Yoann Congal via lists.openembedded.org wrote:
> On Mon Mar 9, 2026 at 3:56 PM CET, Deepak Rathore via lists.openembedded.org wrote:
> > From: Deepak Rathore <deeratho@cisco.com>
> > 
> > Some SDK dependency chains require tcp-wrappers to be available
> > as host/SDK artifacts. The current metadata only partially provides this,
> > which can lead to dependency resolution failures when this recipe is pulled
> > into SDK-oriented builds.
> > 
> > This change does not alter target package behavior; it only enables required
> > native/nativesdk variants for build and SDK integration paths.
> > 
> > Signed-off-by: Deepak Rathore <deeratho@cisco.com>
> > 
> > diff --git a/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb b/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
> > index bcd1d6f792..26eaeab1ac 100644
> > --- a/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
> > +++ b/meta/recipes-extended/tcp-wrappers/tcp-wrappers_7.6.bb
> > @@ -132,3 +132,4 @@ do_install () {
> >  }
> >  
> >  FILES:${PN} += "${sysconfdir}/hosts.allow ${sysconfdir}/hosts.deny"
> > +BBCLASSEXTEND = "native nativesdk"
> 
> That effectively create 2 new recipes tcp-wrappers-native and
> nativesdk-tcp-wrappers. That is not a trivial change that I can take
> without a clear and strong justification.
> 
> What "dependency resolution failures" did you see?
> 
> Also, can you check that tcp-wrapper-native builds on Fedora 41? This is
> the scarthgap supported distro with the most recent GCC : gcc-14 (and it
> usually trigger build failures on old/unmaintained packages)
> 
> Finally, OE-Core master did remove the recipe for it being really
> unmaintained[0]. This is not a good sign :/

The other question that needs to be asked is why would you even need
tcp-wrappers-native? Why are you creating TCP services using the native
tools in a build? It doesn't make any sense.

We've had a flood of these "extend for native/nativesdk" patches and
I'm afraid I'm leaning toward saying no to all of them until we better
understand the actual use cases and make sure this isn't just a random
dependency explosion as people haven't actually thought about the
dependencies.

Cheers,

Richard


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

end of thread, other threads:[~2026-03-18 16:55 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-09 14:56 [openembedded-core][scarthgap][PATCH] tcp-wrappers: extend recipe for native/nativesdk builds Deepak Rathore -X (deeratho - E INFOCHIPS PRIVATE LIMITED at Cisco)
2026-03-18 16:51 ` [OE-core] " Yoann Congal
2026-03-18 16:55   ` Richard Purdie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox