public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: "Luca Boccassi" <luca.boccassi@microsoft.com>
To: "richard.purdie@linuxfoundation.org"
	<richard.purdie@linuxfoundation.org>,
	"openembedded-core@lists.openembedded.org"
	<openembedded-core@lists.openembedded.org>
Cc: "ross.burton@arm.com" <ross.burton@arm.com>
Subject: Re: [OE-core] [PATCH v2] dbus: split -common and -tools out of main package
Date: Thu, 12 Nov 2020 17:53:17 +0000	[thread overview]
Message-ID: <434b42dd8366ea2d24c4bf8332c09f5e7cda45b7.camel@microsoft.com> (raw)
In-Reply-To: <6af65017cec256407db0319271c648275a18ff5d.camel@linuxfoundation.org>

On Thu, 2020-11-12 at 17:29 +0000, Richard Purdie wrote:
> On Mon, 2020-11-02 at 19:18 +0000, Luca Bocassi wrote:
> > From: Luca Boccassi <luca.boccassi@microsoft.com>
> > 
> > Certain config files and units are shared between dbus-daemon and
> > dbus-broker (available in meta-openembedded), so split them out to
> > allow installing dbus-broker without pulling in dbus-daemon and its
> > dependencies.
> > Stand-alone command line tools can also be used regardless of whether
> > the buses are provided by dbus-daemon or dbus-broker, so split them
> > out into dbus-tools.
> > Finally, move the XML schema files out of the main package and into
> > the development package.
> > 
> > All these changes follow the same pattern used by Fedora, which was
> > one of the first distro to switch to dbus-broker by default:
> > 
> > https://src.fedoraproject.org/rpms/dbus/blob/master/f/dbus.spec
> > 
> > Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>
> > ---
> > v2: fix FILES_ regex to correctly include all units but the sockets
> >     in the main package
> 
> I tried putting this through testing on the autobuilder but it fails in
> world builds:
> 
> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1589/steps/13/logs/stdio
> 
> (dbus-native seems to depend upon dbus-native-tools-native)
> 
> so I've had to drop the patch.
> 
> Cheers,
> 
> Richard

Hi,

That's a weird error? Building the basic poky reference didn't show it.

Any suggestion on how to fix it? Do you see anything obviously wrong
with the patch? Is this the correct way to add a runtime dependency in
Yocto:

RDEPENDS_${PN} += "${PN}-common ${PN}-tools"

Should it hard-code the name and drop the ${PN} perhaps?

-- 
Kind regards,
Luca Boccassi

  reply	other threads:[~2020-11-12 17:53 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-02 18:05 [PATCH] dbus: split -common and -tools out of main package Luca Bocassi
2020-11-02 19:18 ` [PATCH v2] " Luca Bocassi
2020-11-11 10:32   ` Luca Boccassi
2020-11-12 17:29   ` [OE-core] " Richard Purdie
2020-11-12 17:53     ` Luca Boccassi [this message]
2020-11-13 10:47 ` [PATCH v3] " Luca Bocassi
2020-11-13 14:34   ` Richard Purdie
2020-11-13 14:50     ` Luca Bocassi
2020-11-13 14:50 ` [PATCH v4] " Luca Bocassi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=434b42dd8366ea2d24c4bf8332c09f5e7cda45b7.camel@microsoft.com \
    --to=luca.boccassi@microsoft.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=richard.purdie@linuxfoundation.org \
    --cc=ross.burton@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox