* [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