Skip to contentSkip to conversation

O.S. Systems' systemd related changes

PATCH In review Otavio Salvador 3 participants 9 messages Feb 7, 2012 – Feb 9, 2012

lore
Share & tools
lore

Otavio Salvador

author
#0 lore raw

cover O.S. Systems' systemd related changes

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
Signature · 1 line
1.7.2.5
author
#1 lore raw

PATCH 1/3 Revert "systemd.bbclass: depends on systemd"

This reverts commit c029d4a268de1ac7076fbeb9e337e96fdb1b226d.

---
 meta-oe/classes/systemd.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
meta-oe/classes/systemd.bbclass +1 -1
index 094a12c..2ca7bfc 100644
@@ -1,4 +1,4 @@
-DEPENDS_append = " systemd systemd-systemctl-native"
+DEPENDS_append = " systemd-systemctl-native"
 
 systemd_postinst() {
 OPTS=""
Signature · 1 line
1.7.2.5
Acked-by on v1 1/3
#2 lore raw

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

Quoted 1 line · Otavio Salvador

This reverts commit c029d4a268de1ac7076fbeb9e337e96fdb1b226d.

Acked-by Martin Jansa <[email protected]>
Quoted 22 lines
---
 meta-oe/classes/systemd.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
meta-oe/classes/systemd.bbclass +1 -1
index 094a12c..2ca7bfc 100644
@@ -1,4 +1,4 @@
-DEPENDS_append = " systemd systemd-systemctl-native"
+DEPENDS_append = " systemd-systemctl-native"
 
 systemd_postinst() {
 OPTS=""
Signature · 7 lines
1.7.2.5


_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]
author
#3 lore raw

PATCH 2/3 udisks: explicitly depends on systemd

In reply to Otavio Salvador
Signed-off-by Otavio Salvador <[email protected]>
---
 meta-oe/recipes-support/udisks/udisks_1.0.4.bb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
meta-oe/recipes-support/udisks/udisks_1.0.4.bb +2 -2
index 55b115e..9ffaca5 100644
@@ -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
Signature · 1 line
1.7.2.5
Acked-by on v1 2/3
#4 lore raw

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

Quoted 1 line · Otavio Salvador
Signed-off-by Otavio Salvador <[email protected]>
Acked-by Martin Jansa <[email protected]>
Quoted 34 lines
---
 meta-oe/recipes-support/udisks/udisks_1.0.4.bb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
meta-oe/recipes-support/udisks/udisks_1.0.4.bb +2 -2
index 55b115e..9ffaca5 100644
@@ -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
Signature · 7 lines
1.7.2.5


_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]
author
#5 lore raw

PATCH 3/3 polkit: depends on systemd

In reply to Otavio Salvador
Signed-off-by Otavio Salvador <[email protected]>
---
 .../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
meta-oe/recipes-extended/polkit/polkit_0.104.bbappend new file +3 -0
new file mode 100644
index 0000000..3c43970
@@ -0,0 +1,3 @@
+DEPENDS += "systemd"
+
+PRINC = "1"
Signature · 1 line
1.7.2.5
#6 lore raw

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

Quoted 1 line · Otavio Salvador
Signed-off-by Otavio Salvador <[email protected]>

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,

Quoted 22 lines
---
 .../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
meta-oe/recipes-extended/polkit/polkit_0.104.bbappend new file +3 -0
new file mode 100644
index 0000000..3c43970
@@ -0,0 +1,3 @@
+DEPENDS += "systemd"
+
+PRINC = "1"
Signature · 7 lines
1.7.2.5


_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]
author
#7 lore raw

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

Quoted 4 lines · Martin Jansa

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?

Signature · 3 lines
Otavio Salvador                             O.S. Systems
E-mail: [email protected]  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
#8 lore raw

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

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

Op 09-02-12 17:14, Otavio Salvador schreef:

Quoted 10 lines
Quoted 3 lines · Martin Jansa

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-----