[PATCH][meta-oe 0/3] O.S. Systems' systemd related changes

[PATCH][meta-oe 0/3] O.S. Systems' systemd related changes

[Otavio Salvador]

The following changes since commit 724ec1ea4ff52b88bf1f6970557e44768214fd7b:

  systemd: move to v40 and don't kill user processes on exit (2012-02-07 16:06:19 +0100)

are available in the git repository at:
  git://github.com/OSSystems/meta-oe master
  https://github.com/OSSystems/meta-oe/tree/master

Otavio Salvador (3):
  Revert "systemd.bbclass: depends on systemd"
  udisks: explicitly depends on systemd
  polkit: depends on systemd

 meta-oe/classes/systemd.bbclass                    |    2 +-
 .../recipes-extended/polkit/polkit_0.104.bbappend  |    3 +++
 meta-oe/recipes-support/udisks/udisks_1.0.4.bb     |    4 ++--
 3 files changed, 6 insertions(+), 3 deletions(-)
 create mode 100644 meta-oe/recipes-extended/polkit/polkit_0.104.bbappend

-- 
1.7.2.5

[PATCH][meta-oe 1/3] Revert "systemd.bbclass: depends on systemd"

[Otavio Salvador]

This reverts commit c029d4a268de1ac7076fbeb9e337e96fdb1b226d.
---
 meta-oe/classes/systemd.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta-oe/classes/systemd.bbclass b/meta-oe/classes/systemd.bbclass
index 094a12c..2ca7bfc 100644
--- a/meta-oe/classes/systemd.bbclass
+++ b/meta-oe/classes/systemd.bbclass
@@ -1,4 +1,4 @@
-DEPENDS_append = " systemd systemd-systemctl-native"
+DEPENDS_append = " systemd-systemctl-native"
 
 systemd_postinst() {
 OPTS=""
-- 
1.7.2.5

[PATCH][meta-oe 2/3] udisks: explicitly depends on systemd

[Otavio Salvador]

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 meta-oe/recipes-support/udisks/udisks_1.0.4.bb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-support/udisks/udisks_1.0.4.bb b/meta-oe/recipes-support/udisks/udisks_1.0.4.bb
index 55b115e..9ffaca5 100644
--- a/meta-oe/recipes-support/udisks/udisks_1.0.4.bb
+++ b/meta-oe/recipes-support/udisks/udisks_1.0.4.bb
@@ -2,7 +2,7 @@ DESCRIPTION = "A storage daemon that implements well-defined D-Bus interfaces th
 LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=73d83aebe7e4b62346afde80e0e94273"
 
-DEPENDS = "libatasmart sg3-utils polkit udev dbus-glib glib-2.0"
+DEPENDS = "libatasmart sg3-utils polkit udev dbus-glib glib-2.0 systemd"
 # optional dependencies: device-mapper parted
 
 SRC_URI = "http://hal.freedesktop.org/releases/${BPN}-${PV}.tar.gz;name=${BPN} \
@@ -12,7 +12,7 @@ SRC_URI = "http://hal.freedesktop.org/releases/${BPN}-${PV}.tar.gz;name=${BPN} \
 SRC_URI[udisks.md5sum] = "86c63b2b5484f2060499a052b5b6256b"
 SRC_URI[udisks.sha256sum] = "854b89368733b9c3a577101b761ad5397ae75a05110c8698ac5b29de9a8bf8f5"
 
-PR = "r2"
+PR = "r3"
 
 inherit autotools systemd
 
-- 
1.7.2.5

[PATCH][meta-oe 3/3] polkit: depends on systemd

[Otavio Salvador]

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 .../recipes-extended/polkit/polkit_0.104.bbappend  |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
 create mode 100644 meta-oe/recipes-extended/polkit/polkit_0.104.bbappend

diff --git a/meta-oe/recipes-extended/polkit/polkit_0.104.bbappend b/meta-oe/recipes-extended/polkit/polkit_0.104.bbappend
new file mode 100644
index 0000000..3c43970
--- /dev/null
+++ b/meta-oe/recipes-extended/polkit/polkit_0.104.bbappend
@@ -0,0 +1,3 @@
+DEPENDS += "systemd"
+
+PRINC = "1"
-- 
1.7.2.5

Re: [PATCH][meta-oe 3/3] polkit: depends on systemd

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 1616 bytes --]

On Tue, Feb 07, 2012 at 08:58:24PM +0000, Otavio Salvador wrote:
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Can you please change it to something like:

SESSION_EXTRA_DEPENDS = "systemd"
SESSION_EXTRA_OECONF = "--enable-systemd"

DEPENDS += "${SESSION_EXTRA_DEPENDS}"
EXTRA_OECONF += "${SESSION_EXTRA_OECONF}"

PRINC := "${@int(PRINC) + 1}"

So it's clear that systemd will always be enabled and still easy to override
both with .bbappend?

With library split from:
http://lists.linuxtogo.org/pipermail/openembedded-devel/2012-February/037828.html
it won't be so big problem as described here:
http://lists.linuxtogo.org/pipermail/openembedded-devel/2012-January/037597.html
but still better to make it easy to change this in some upper layer

Cheers,

> ---
>  .../recipes-extended/polkit/polkit_0.104.bbappend  |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>  create mode 100644 meta-oe/recipes-extended/polkit/polkit_0.104.bbappend
> 
> diff --git a/meta-oe/recipes-extended/polkit/polkit_0.104.bbappend b/meta-oe/recipes-extended/polkit/polkit_0.104.bbappend
> new file mode 100644
> index 0000000..3c43970
> --- /dev/null
> +++ b/meta-oe/recipes-extended/polkit/polkit_0.104.bbappend
> @@ -0,0 +1,3 @@
> +DEPENDS += "systemd"
> +
> +PRINC = "1"
> -- 
> 1.7.2.5
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH][meta-oe 1/3] Revert "systemd.bbclass: depends on systemd"

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 952 bytes --]

On Tue, Feb 07, 2012 at 08:58:22PM +0000, Otavio Salvador wrote:
> This reverts commit c029d4a268de1ac7076fbeb9e337e96fdb1b226d.

Acked-by: Martin Jansa <Martin.Jansa@gmail.com>

> ---
>  meta-oe/classes/systemd.bbclass |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/meta-oe/classes/systemd.bbclass b/meta-oe/classes/systemd.bbclass
> index 094a12c..2ca7bfc 100644
> --- a/meta-oe/classes/systemd.bbclass
> +++ b/meta-oe/classes/systemd.bbclass
> @@ -1,4 +1,4 @@
> -DEPENDS_append = " systemd systemd-systemctl-native"
> +DEPENDS_append = " systemd-systemctl-native"
>  
>  systemd_postinst() {
>  OPTS=""
> -- 
> 1.7.2.5
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH][meta-oe 2/3] udisks: explicitly depends on systemd

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 1686 bytes --]

On Tue, Feb 07, 2012 at 08:58:23PM +0000, Otavio Salvador wrote:
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Acked-by: Martin Jansa <Martin.Jansa@gmail.com>

> ---
>  meta-oe/recipes-support/udisks/udisks_1.0.4.bb |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-oe/recipes-support/udisks/udisks_1.0.4.bb b/meta-oe/recipes-support/udisks/udisks_1.0.4.bb
> index 55b115e..9ffaca5 100644
> --- a/meta-oe/recipes-support/udisks/udisks_1.0.4.bb
> +++ b/meta-oe/recipes-support/udisks/udisks_1.0.4.bb
> @@ -2,7 +2,7 @@ DESCRIPTION = "A storage daemon that implements well-defined D-Bus interfaces th
>  LICENSE = "GPLv2+"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=73d83aebe7e4b62346afde80e0e94273"
>  
> -DEPENDS = "libatasmart sg3-utils polkit udev dbus-glib glib-2.0"
> +DEPENDS = "libatasmart sg3-utils polkit udev dbus-glib glib-2.0 systemd"
>  # optional dependencies: device-mapper parted
>  
>  SRC_URI = "http://hal.freedesktop.org/releases/${BPN}-${PV}.tar.gz;name=${BPN} \
> @@ -12,7 +12,7 @@ SRC_URI = "http://hal.freedesktop.org/releases/${BPN}-${PV}.tar.gz;name=${BPN} \
>  SRC_URI[udisks.md5sum] = "86c63b2b5484f2060499a052b5b6256b"
>  SRC_URI[udisks.sha256sum] = "854b89368733b9c3a577101b761ad5397ae75a05110c8698ac5b29de9a8bf8f5"
>  
> -PR = "r2"
> +PR = "r3"
>  
>  inherit autotools systemd
>  
> -- 
> 1.7.2.5
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH][meta-oe 3/3] polkit: depends on systemd

[Otavio Salvador]

On Thu, Feb 9, 2012 at 14:00, Martin Jansa <martin.jansa@gmail.com> wrote:

> So it's clear that systemd will always be enabled and still easy to
> override
> both with .bbappend?
>

For the current state of polkit it might be better to disable systemd on
this, completely. So passing --disable-systemd to autoconf. What do you
think?

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Re: [PATCH][meta-oe 3/3] polkit: depends on systemd

[Koen Kooi]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Op 09-02-12 17:14, Otavio Salvador schreef:
> On Thu, Feb 9, 2012 at 14:00, Martin Jansa <martin.jansa@gmail.com>
> wrote:
> 
>> So it's clear that systemd will always be enabled and still easy to 
>> override both with .bbappend?
>> 
> 
> For the current state of polkit it might be better to disable systemd on 
> this, completely. So passing --disable-systemd to autoconf. What do you 
> think?

Well, systemd is the only backend that actually works in our current
metadata, so I'm hesitant to globally disable it.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (Darwin)
Comment: GPGTools - http://gpgtools.org

iEYEARECAAYFAk80FbEACgkQMkyGM64RGpFyNQCfU0ANmYwoxrK4+J8bzMGr8Wy3
J7sAnil8V8UOxuc+p+8zCXszOB8zhbzp
=/tbB
-----END PGP SIGNATURE-----