public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* [OE-core][scarthgap][PATCH v1] libevent: merge duplicate inherit statements
@ 2026-02-09 12:42 Pratik Farkase
  2026-02-27 21:52 ` Yoann Congal
  0 siblings, 1 reply; 2+ messages in thread
From: Pratik Farkase @ 2026-02-09 12:42 UTC (permalink / raw)
  To: openembedded-core; +Cc: pratik.farkase, Pratik Farkase

Merge the separate inherit statements into a single one for
improved readability and consistency with OE style.

No functional change intended.

Signed-off-by: Pratik Farkase <pratik.farkase@est.tech>
---
 meta/recipes-support/libevent/libevent_2.1.12.bb | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta/recipes-support/libevent/libevent_2.1.12.bb b/meta/recipes-support/libevent/libevent_2.1.12.bb
index 25388fb4d7..58502edc32 100644
--- a/meta/recipes-support/libevent/libevent_2.1.12.bb
+++ b/meta/recipes-support/libevent/libevent_2.1.12.bb
@@ -28,13 +28,11 @@ S = "${WORKDIR}/${BPN}-${PV}-stable"
 PACKAGECONFIG ??= ""
 PACKAGECONFIG[openssl] = "--enable-openssl,--disable-openssl,openssl"
 
-inherit autotools github-releases
+inherit autotools github-releases ptest multilib_header
 
 # Needed for Debian packaging
 LEAD_SONAME = "libevent-2.1.so"
 
-inherit ptest multilib_header
-
 DEPENDS = "zlib"
 
 PACKAGES_DYNAMIC = "^${PN}-.*$"
-- 
2.43.0



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

* Re: [OE-core][scarthgap][PATCH v1] libevent: merge duplicate inherit statements
  2026-02-09 12:42 [OE-core][scarthgap][PATCH v1] libevent: merge duplicate inherit statements Pratik Farkase
@ 2026-02-27 21:52 ` Yoann Congal
  0 siblings, 0 replies; 2+ messages in thread
From: Yoann Congal @ 2026-02-27 21:52 UTC (permalink / raw)
  To: pratik.farkase, openembedded-core; +Cc: pratik.farkase

Hello,

On Mon Feb 9, 2026 at 1:42 PM CET, Pratik Farkase via lists.openembedded.org wrote:
> Merge the separate inherit statements into a single one for
> improved readability and consistency with OE style.
>
> No functional change intended.
>
> Signed-off-by: Pratik Farkase <pratik.farkase@est.tech>

Sorry, but we ultimately did not take this patch for stable inclusion:
This is only a cleanup and there is little to no advantage to merge
this.

Regards,
> ---
>  meta/recipes-support/libevent/libevent_2.1.12.bb | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/meta/recipes-support/libevent/libevent_2.1.12.bb b/meta/recipes-support/libevent/libevent_2.1.12.bb
> index 25388fb4d7..58502edc32 100644
> --- a/meta/recipes-support/libevent/libevent_2.1.12.bb
> +++ b/meta/recipes-support/libevent/libevent_2.1.12.bb
> @@ -28,13 +28,11 @@ S = "${WORKDIR}/${BPN}-${PV}-stable"
>  PACKAGECONFIG ??= ""
>  PACKAGECONFIG[openssl] = "--enable-openssl,--disable-openssl,openssl"
>  
> -inherit autotools github-releases
> +inherit autotools github-releases ptest multilib_header
>  
>  # Needed for Debian packaging
>  LEAD_SONAME = "libevent-2.1.so"
>  
> -inherit ptest multilib_header
> -
>  DEPENDS = "zlib"
>  
>  PACKAGES_DYNAMIC = "^${PN}-.*$"
-- 
Yoann Congal
Smile ECS



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

end of thread, other threads:[~2026-02-27 21:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-09 12:42 [OE-core][scarthgap][PATCH v1] libevent: merge duplicate inherit statements Pratik Farkase
2026-02-27 21:52 ` Yoann Congal

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