diff mbox series

[walnascar,1/7] default-providers: Add VIRTUAL-RUNTIME_dbus variable

Message ID 20250618124256.3261625-1-praveen.kumar@windriver.com
State New
Headers show
Series [walnascar,1/7] default-providers: Add VIRTUAL-RUNTIME_dbus variable | expand

Commit Message

Praveen Kumar June 18, 2025, 12:42 p.m. UTC
From: Niko Mauno <niko.mauno@vaisala.com>

The purpose of the new variable is to facilitate oe-core users who wish
to use an alternative runtime D-Bus implementation instead of the
default Freedesktop.org's dbus, such as dbus-broker, a recipe for which
is currently available under
meta-openembedded/meta-oe/recipes-core/dbus/

While introducing this facilitation the intent is to preserve the
existing functionality, while allowing the user to optionally select an
alternative runtime D-Bus implementation by adding the following line
e.g. to local.conf file:

  VIRTUAL-RUNTIME_dbus = "dbus-broker"

As a background, for example the Fedora distribution uses dbus-broker
instead of Freedesktop.org's D-Bus implementation. The following
excerpts from
https://fedoraproject.org/wiki/Changes/DbusBrokerAsTheDefaultDbusImplementation
provide background for their technological decision

  This change provides a more scalable and more reliable implementation in place of the reference implementation.
  The reference implementation suffers from long-standing issues including potential dead-locks and susceptibility to denial of service attacks, which the replacement implementation does not.

and

  No visible changes in behavior are expected, except for different log messages.
  Any visibly different behavior to `dbus-daemon` should be reported as a regression.
  Ideally, this change should only improve the performance and security of the message bus.

(From OE-Core rev: 81fd917ac05be19d8345fff272a2ffc17a257880)

Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Praveen Kumar <praveen.kumar@windriver.com>
---
 meta/conf/distro/include/default-providers.inc | 1 +
 1 file changed, 1 insertion(+)

Comments

Randy MacLeod June 18, 2025, 6:39 p.m. UTC | #1
Add Steve who maintains walnascar.


On 2025-06-18 8:42 a.m., Praveen Kumar via lists.openembedded.org wrote:
> From: Niko Mauno<niko.mauno@vaisala.com>
>
> The purpose of the new variable is to facilitate oe-core users who wish
> to use an alternative runtime D-Bus implementation instead of the
> default Freedesktop.org's dbus, such as dbus-broker, a recipe for which
> is currently available under
> meta-openembedded/meta-oe/recipes-core/dbus/
>
> While introducing this facilitation the intent is to preserve the
> existing functionality, while allowing the user to optionally select an
> alternative runtime D-Bus implementation by adding the following line
> e.g. to local.conf file:
>
>    VIRTUAL-RUNTIME_dbus = "dbus-broker"
>
> As a background, for example the Fedora distribution uses dbus-broker
> instead of Freedesktop.org's D-Bus implementation. The following
> excerpts from
> https://fedoraproject.org/wiki/Changes/DbusBrokerAsTheDefaultDbusImplementation
> provide background for their technological decision
>
>    This change provides a more scalable and more reliable implementation in place of the reference implementation.
>    The reference implementation suffers from long-standing issues including potential dead-locks and susceptibility to denial of service attacks, which the replacement implementation does not.
>
> and
>
>    No visible changes in behavior are expected, except for different log messages.
>    Any visibly different behavior to `dbus-daemon` should be reported as a regression.
>    Ideally, this change should only improve the performance and security of the message bus.
>
> (From OE-Core rev: 81fd917ac05be19d8345fff272a2ffc17a257880)
>
> Signed-off-by: Niko Mauno<niko.mauno@vaisala.com>
> Signed-off-by: Richard Purdie<richard.purdie@linuxfoundation.org>
> Signed-off-by: Praveen Kumar<praveen.kumar@windriver.com>
> ---
>   meta/conf/distro/include/default-providers.inc | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/meta/conf/distro/include/default-providers.inc b/meta/conf/distro/include/default-providers.inc
> index d3eefb754b..cebd0400f4 100644
> --- a/meta/conf/distro/include/default-providers.inc
> +++ b/meta/conf/distro/include/default-providers.inc
> @@ -31,6 +31,7 @@ VIRTUAL-RUNTIME_base-utils ?= "busybox"
>   VIRTUAL-RUNTIME_base-utils-hwclock ?= "busybox-hwclock"
>   VIRTUAL-RUNTIME_base-utils-syslog ?= "busybox-syslog"
>   VIRTUAL-RUNTIME_keymaps ?= "keymaps"
> +VIRTUAL-RUNTIME_dbus ?= "dbus"

Praveen,

This change-set borders on new feature development in a released branch.

Unless you have a very good use case, we'll likely reject it the 
backport to walnascar.

Can you explain why you want / need the backport in a reply to this 
thread and
if you send a v2, in a cover letter to the v2 series.

Thanks,


../Randy



>   
>   #
>   # Default recipe providers
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#219000):https://lists.openembedded.org/g/openembedded-core/message/219000
> Mute This Topic:https://lists.openembedded.org/mt/113707135/3616765
> Group Owner:openembedded-core+owner@lists.openembedded.org
> Unsubscribe:https://lists.openembedded.org/g/openembedded-core/unsub [randy.macleod@windriver.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta/conf/distro/include/default-providers.inc b/meta/conf/distro/include/default-providers.inc
index d3eefb754b..cebd0400f4 100644
--- a/meta/conf/distro/include/default-providers.inc
+++ b/meta/conf/distro/include/default-providers.inc
@@ -31,6 +31,7 @@  VIRTUAL-RUNTIME_base-utils ?= "busybox"
 VIRTUAL-RUNTIME_base-utils-hwclock ?= "busybox-hwclock"
 VIRTUAL-RUNTIME_base-utils-syslog ?= "busybox-syslog"
 VIRTUAL-RUNTIME_keymaps ?= "keymaps"
+VIRTUAL-RUNTIME_dbus ?= "dbus"
 
 #
 # Default recipe providers