All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [meta-networking][PATCH] networkingmanager: QA warning for rdepends on polkit
Date: Mon, 11 May 2015 11:09:52 +0200	[thread overview]
Message-ID: <20150511090952.GF2400@jama> (raw)
In-Reply-To: <1430795900-25986-1-git-send-email-akuster808@gmail.com>

On Mon, May 04, 2015 at 08:18:20PM -0700, Armin Kuster wrote:
> WARNING: QA Issue: networkmanager rdepends on polkit, but it isn't a build dependency? [build-deps]
> 
> Signed-off-by: Armin Kuster <akuster808@gmail.com>
> ---
>  .../recipes-connectivity/networkmanager/networkmanager_0.9.8.10.bb     | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-networking/recipes-connectivity/networkmanager/networkmanager_0.9.8.10.bb b/meta-networking/recipes-connectivity/networkmanager/networkmanager_0.9.8.10.bb
> index 8d37c98..36a4a7e 100644
> --- a/meta-networking/recipes-connectivity/networkmanager/networkmanager_0.9.8.10.bb
> +++ b/meta-networking/recipes-connectivity/networkmanager/networkmanager_0.9.8.10.bb
> @@ -37,11 +37,12 @@ PACKAGECONFIG ??= "${@base_contains('DISTRO_FEATURES','systemd','systemd','conso
>  PACKAGECONFIG[systemd] = " \
>      --with-systemdsystemunitdir=${systemd_unitdir}/system --with-session-tracking=systemd --enable-polkit, \
>      --without-systemdsystemunitdir, \
> -    polkit \
> +    polkit, polkit \
>  "
>  # consolekit is not picked by shlibs, so add it to RDEPENDS too
>  PACKAGECONFIG[consolekit] = "--with-session-tracking=consolekit,,consolekit,consolekit"
>  PACKAGECONFIG[concheck] = "--enable-concheck,--disable-concheck,libsoup-2.4"
> +PACKAGECONFIG[polkit] = "--enable-polkit, --disable-polkit, , polkit"

Shouldn't it have polkit as build time dependency (3rd argument, not 4th)

Also the subject is wrong, it's networkmanager, not networkingmanager.

>  
>  # Work around dbus permission problems since we lack a proper at_console
>  do_install_prepend() {
> -- 
> 2.3.5
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com


  parent reply	other threads:[~2015-05-11  9:09 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-05  3:18 [meta-networking][PATCH] networkingmanager: QA warning for rdepends on polkit Armin Kuster
2015-05-08 15:12 ` Joe MacDonald
2015-05-08 17:02   ` akuster808
2015-05-08 17:18     ` Joe MacDonald
2015-05-11  9:09 ` Martin Jansa [this message]
2015-05-11 18:44   ` Khem Raj
2015-05-11 19:00     ` Martin Jansa
2015-05-11 19:43       ` Khem Raj
2015-05-11 19:46         ` Khem Raj
2015-05-11 19:56           ` Martin Jansa
2015-05-11 19:58           ` akuster808
2015-05-11 19:52         ` Martin Jansa
2015-05-12  8:53           ` Paul Eggleton

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=20150511090952.GF2400@jama \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.