Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Adrian Freihofer <adrian.freihofer@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH 20/34] networkmanager: fix introspection and vala support
Date: Sun, 28 Feb 2016 14:10:11 +0100	[thread overview]
Message-ID: <1456665011.2775.24.camel@gmail.com> (raw)
In-Reply-To: <db2af35a3af2a5e9a18611b66d05c8b91c0e73e5.1456236552.git.alexander.kanavin@linux.intel.com>

Hi Alexander

Please consider the following lines before adding polkit to DEPENDS:
PACKAGECONFIG[systemd] = " \
    --with-systemdsystemunitdir=${systemd_unitdir}/system --with-session-tracking=systemd --enable-polkit, \
    --without-systemdsystemunitdir, \
    polkit \
"

I think the final solution should be a flag on DISTRO_FEATURES level to enable/disable polkit for systemd, networkmanager, modemmanager and other packages.
As a first step you might split polkit PACKAGECONFIG from systemd PACKAGECONFIG. The (untested) patch allows to enable disable polkit independently from systemd without changing
the default settings.

> diff --git a/meta-oe/recipes-connectivity/networkmanager/networkmanager_1.0.10.bb b/meta-oe/recipes-connectivity/networkmanager/networkmanager_1.0.10.bb
> index b00c9bb..1e64c4b 100644
> --- a/meta-oe/recipes-connectivity/networkmanager/networkmanager_1.0.10.bb
> +++ b/meta-oe/recipes-connectivity/networkmanager/networkmanager_1.0.10.bb
> @@ -35,15 +35,15 @@ EXTRA_OECONF = " \
>  "
>  
>  PACKAGECONFIG ??= "nss ifupdown netconfig dhclient dnsmasq \
> -    ${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd','consolekit',d)} \
> +    ${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd polkit','consolekit',d)} \
>      ${@bb.utils.contains('DISTRO_FEATURES','bluetooth','${BLUEZ}','',d)} \
>      ${@bb.utils.contains('DISTRO_FEATURES','wifi','wifi','',d)} \
>  "
>  PACKAGECONFIG[systemd] = " \
> -    --with-systemdsystemunitdir=${systemd_unitdir}/system --with-session-tracking=systemd --enable-polkit, \
> -    --without-systemdsystemunitdir, \
> -    polkit \
> +    --with-systemdsystemunitdir=${systemd_unitdir}/system --with-session-tracking=systemd, \
> +    --without-systemdsystemunitdir \
>  "
> +PACKAGECONFIG[polkit] = "--enable-polkit, --disable-polkit, polkit"
>  PACKAGECONFIG[bluez5] = "--enable-bluez5-dun,--disable-bluez5-dun,bluez5"
>  # consolekit is not picked by shlibs, so add it to RDEPENDS too
>  PACKAGECONFIG[consolekit] = "--with-session-tracking=consolekit,,consolekit,consolekit"

Regards,
Adrian


  reply	other threads:[~2016-02-28 13:10 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-23 14:19 [PATCH 00/34] Initial gobject introspection support for meta-openembedded Alexander Kanavin
2016-02-23 14:19 ` [PATCH 01/34] evolution-data-server: fix gobject introspection Alexander Kanavin
2016-02-23 14:19 ` [PATCH 02/34] evince: update to 3.18.2 Alexander Kanavin
2016-02-23 14:19 ` [PATCH 03/34] gnome-bluetooth: " Alexander Kanavin
2016-02-23 14:19 ` [PATCH 04/34] gnome-desktop: remove pygtk dependency Alexander Kanavin
2016-02-23 14:19 ` [PATCH 05/34] gnumeric: fix gobject introspection support Alexander Kanavin
2016-02-23 14:19 ` [PATCH 06/34] libgdata: update to 0.16.1 Alexander Kanavin
2016-02-23 14:19 ` [PATCH 07/34] libgsf: fix gobject introspection Alexander Kanavin
2016-02-23 14:19 ` [PATCH 08/34] libwnck: " Alexander Kanavin
2016-02-23 14:19 ` [PATCH 09/34] libxklavier: " Alexander Kanavin
2016-02-23 14:19 ` [PATCH 10/34] goffice: " Alexander Kanavin
2016-02-23 14:19 ` [PATCH 11/34] tracker: " Alexander Kanavin
2016-02-24 13:55   ` Martin Jansa
2016-02-23 14:19 ` [PATCH 12/34] gupnp: " Alexander Kanavin
2016-02-23 14:19 ` [PATCH 13/34] rygel: " Alexander Kanavin
2016-02-23 14:19 ` [PATCH 14/34] media-service-demo: remove the recipe Alexander Kanavin
2016-02-23 14:19 ` [PATCH 15/34] python-gst: remove the 0.10 recipe Alexander Kanavin
2016-02-23 14:19 ` [PATCH 16/34] gstreamer-0.10: disable gobject introspection Alexander Kanavin
2016-02-23 14:19 ` [PATCH 17/34] libmediaart: fix " Alexander Kanavin
2016-02-23 14:19 ` [PATCH 18/34] bluez4: use real introspection instead of legacy pygobject Alexander Kanavin
2016-02-23 14:19 ` [PATCH 19/34] modemmanager: fix vala support Alexander Kanavin
2016-02-23 14:19 ` [PATCH 20/34] networkmanager: fix introspection and " Alexander Kanavin
2016-02-28 13:10   ` Adrian Freihofer [this message]
2016-02-23 14:20 ` [PATCH 21/34] telepathy-glib: fix gobject introspection support Alexander Kanavin
2016-02-23 14:20 ` [PATCH 22/34] libgee: " Alexander Kanavin
2016-02-23 14:20 ` [PATCH 23/34] polkit: " Alexander Kanavin
2016-02-23 14:20 ` [PATCH 24/34] gtkextra: " Alexander Kanavin
2016-02-23 14:20 ` [PATCH 25/34] libunique: " Alexander Kanavin
2016-02-23 14:20 ` [PATCH 26/34] babl: update to 0.1.16 Alexander Kanavin
2016-02-23 14:20 ` [PATCH 27/34] gegl: update to 0.3.4 Alexander Kanavin
2016-02-23 14:20 ` [PATCH 28/34] gimp: accept newer versins of babl and gegl Alexander Kanavin
2016-02-23 14:20 ` [PATCH 29/34] omgps: remove the recipe Alexander Kanavin
2016-02-23 14:20 ` [PATCH 30/34] poppler-data: fix do_install() Alexander Kanavin
2016-02-24 12:55   ` Martin Jansa
2016-02-23 14:20 ` [PATCH 31/34] poppler: fix gobject introspection Alexander Kanavin
2016-02-23 14:20 ` [PATCH 32/34] udisks2: " Alexander Kanavin
2016-02-23 14:20 ` [PATCH 33/34] upower: " Alexander Kanavin
2016-02-23 14:20 ` [PATCH 34/34] python-dbusmock: use real introspection instead on legacy pygobject Alexander Kanavin
2016-02-24 16:47 ` [PATCH 00/34] Initial gobject introspection support for meta-openembedded Martin Jansa
2016-02-25 13:36   ` Alexander Kanavin
2016-02-25 14:23     ` Martin Jansa

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=1456665011.2775.24.camel@gmail.com \
    --to=adrian.freihofer@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox