[PATCH 0/2] systemd: SRC_URI fix + package split

[PATCH 0/2] systemd: SRC_URI fix + package split

[Anders Darander]

Systemd's git repo has moved on freedesktop.org, thus fix the SRC_URI.

At least while using rpm-packaging, the systemd package (rpm) got a dependency
on python, as systemd-analyze needs this (in the shebang). When trying to
create a small image using systemd, this isn't always desirable. Thus the
splitting of systemd-analyze to a separate package.



The following changes since commit 9c855444d025a85f28196bfeae25dd48e44510c5:

  xf86-video-fbdev: remove, driver is now in oe-core (2012-01-09 10:31:00 +0100)

are available in the git repository at:
  git://github.com/darander/meta-oe systemd-move
  https://github.com/darander/meta-oe/tree/systemd-move

Anders Darander (2):
  systemd: git repo moved
  systemd: split out systemd-analyze

 meta-oe/recipes-core/systemd/systemd_git.bb |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

-- 
1.7.8.2

[PATCH 1/2] systemd: git repo moved

[Anders Darander]

Freedesktop have reorganized the git repository structure.

Signed-off-by: Anders Darander <anders@chargestorm.se>
---
 meta-oe/recipes-core/systemd/systemd_git.bb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta-oe/recipes-core/systemd/systemd_git.bb b/meta-oe/recipes-core/systemd/systemd_git.bb
index 2548d10..bdaaf16 100644
--- a/meta-oe/recipes-core/systemd/systemd_git.bb
+++ b/meta-oe/recipes-core/systemd/systemd_git.bb
@@ -20,7 +20,7 @@ inherit useradd pkgconfig autotools vala perlnative
 
 SRCREV = "f6cebb3bd5a00d79c8131637c0f6796a75e6af99"
 
-SRC_URI = "git://anongit.freedesktop.org/systemd;protocol=git \
+SRC_URI = "git://anongit.freedesktop.org/systemd/systemd;protocol=git \
            file://0001-disable-xsltproc-completely.patch \
            ${UCLIBCPATCHES} \
           "
-- 
1.7.8.2

[PATCH 2/2] systemd: split out systemd-analyze

[Anders Darander]

Split systemd-analyze to a separate package, to remove the implicit
dependency on python. (At least the rpm-packaging backend created a
run-time dependency on python).

Signed-off-by: Anders Darander <anders@chargestorm.se>
---
 meta-oe/recipes-core/systemd/systemd_git.bb |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-core/systemd/systemd_git.bb b/meta-oe/recipes-core/systemd/systemd_git.bb
index bdaaf16..1d71bb6 100644
--- a/meta-oe/recipes-core/systemd/systemd_git.bb
+++ b/meta-oe/recipes-core/systemd/systemd_git.bb
@@ -14,7 +14,7 @@ inherit gitpkgv
 PKGV = "v${GITPKGVTAG}"
 
 PV = "git"
-PR = "r7"
+PR = "r8"
 
 inherit useradd pkgconfig autotools vala perlnative
 
@@ -57,11 +57,13 @@ do_install() {
 	ln -s ${base_bindir}/systemd ${D}/init
 }
 
-PACKAGES =+ "${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs"
+PACKAGES =+ "${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs ${PN}-analyze"
 
 USERADD_PACKAGES = "${PN}"
 GROUPADD_PARAM_${PN} = "-r lock"
 
+FILES_${PN}-analyze = "${bindir}/systemd-analyze"
+
 FILES_${PN}-initramfs = "/init"
 RDEPENDS_${PN}-initramfs = "${PN}"
 
-- 
1.7.8.2

Re: [PATCH 2/2] systemd: split out systemd-analyze

[Martin Jansa]

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

On Mon, Jan 09, 2012 at 02:18:38PM +0100, Anders Darander wrote:
> Split systemd-analyze to a separate package, to remove the implicit
> dependency on python. (At least the rpm-packaging backend created a
> run-time dependency on python).
> 
> Signed-off-by: Anders Darander <anders@chargestorm.se>
> ---
>  meta-oe/recipes-core/systemd/systemd_git.bb |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-oe/recipes-core/systemd/systemd_git.bb b/meta-oe/recipes-core/systemd/systemd_git.bb
> index bdaaf16..1d71bb6 100644
> --- a/meta-oe/recipes-core/systemd/systemd_git.bb
> +++ b/meta-oe/recipes-core/systemd/systemd_git.bb
> @@ -14,7 +14,7 @@ inherit gitpkgv
>  PKGV = "v${GITPKGVTAG}"
>  
>  PV = "git"
> -PR = "r7"
> +PR = "r8"
>  
>  inherit useradd pkgconfig autotools vala perlnative
>  
> @@ -57,11 +57,13 @@ do_install() {
>  	ln -s ${base_bindir}/systemd ${D}/init
>  }
>  
> -PACKAGES =+ "${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs"
> +PACKAGES =+ "${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs ${PN}-analyze"
>  
>  USERADD_PACKAGES = "${PN}"
>  GROUPADD_PARAM_${PN} = "-r lock"
>  
> +FILES_${PN}-analyze = "${bindir}/systemd-analyze"

according to your commit message, why not add
RDEPENDS_${PN}-analyze = "python"
here?

and maybe
RSUGGESTS_${PN} += "${PN}-analyze"

> +
>  FILES_${PN}-initramfs = "/init"
>  RDEPENDS_${PN}-initramfs = "${PN}"
>  
> -- 
> 1.7.8.2
> 
> 
> _______________________________________________
> 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 2/2] systemd: split out systemd-analyze

[Anders Darander]

* Martin Jansa <martin.jansa@gmail.com> [120109 14:28]:

> On Mon, Jan 09, 2012 at 02:18:38PM +0100, Anders Darander wrote:
> > Split systemd-analyze to a separate package, to remove the implicit
> > dependency on python. (At least the rpm-packaging backend created a
> > run-time dependency on python).

> > +PACKAGES =+ "${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs ${PN}-analyze"

> >  USERADD_PACKAGES = "${PN}"
> >  GROUPADD_PARAM_${PN} = "-r lock"

> > +FILES_${PN}-analyze = "${bindir}/systemd-analyze"

> according to your commit message, why not add
> RDEPENDS_${PN}-analyze = "python"
> here?

Yes, I should probably spin a v2 with:
RDEPENDS_${PN}-analyze = "python-dbus"
as python alone isn't enough to run systemd-analyze.

Missing that as on our larger image, python was always explicitly
installed by other packages, and on this smaller one, we didn't want it.
Good catch, though!

> and maybe
> RSUGGESTS_${PN} += "${PN}-analyze"

Not sure about this one. It should probably be safe to add, although I'm
undecided...

Cheers,
Anders

> > +
> >  FILES_${PN}-initramfs = "/init"
> >  RDEPENDS_${PN}-initramfs = "${PN}"
-- 
Anders Darander
ChargeStorm AB / eStorm AB

Re: [PATCH 2/2] systemd: split out systemd-analyze

[Anders Darander]

* Anders Darander <anders@chargestorm.se> [120109 14:37]:
> * Martin Jansa <martin.jansa@gmail.com> [120109 14:28]:

> > On Mon, Jan 09, 2012 at 02:18:38PM +0100, Anders Darander wrote:
> > > Split systemd-analyze to a separate package, to remove the implicit
> > > dependency on python. (At least the rpm-packaging backend created a
> > > run-time dependency on python).

> > > +PACKAGES =+ "${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs ${PN}-analyze"

> > >  USERADD_PACKAGES = "${PN}"
> > >  GROUPADD_PARAM_${PN} = "-r lock"

> > > +FILES_${PN}-analyze = "${bindir}/systemd-analyze"

> > according to your commit message, why not add
> > RDEPENDS_${PN}-analyze = "python"
> > here?

> Yes, I should probably spin a v2 with:
> RDEPENDS_${PN}-analyze = "python-dbus"
> as python alone isn't enough to run systemd-analyze.

Just pushed the above fix to the same systemd-move branch on github.

Cheers,
Anders

> Missing that as on our larger image, python was always explicitly
> installed by other packages, and on this smaller one, we didn't want it.
> Good catch, though!

-- 
Anders Darander
ChargeStorm AB / eStorm AB

Re: [PATCH 2/2] systemd: split out systemd-analyze

[Otavio Salvador]

On Mon, Jan 9, 2012 at 11:44, Anders Darander <anders@chargestorm.se> wrote:

> Just pushed the above fix to the same systemd-move branch on github.
>

Please send an updated patchset so people can review it.

-- 
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 2/2] systemd: split out systemd-analyze

[Koen Kooi]

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

Op 09-01-12 14:37, Anders Darander schreef:
> * Martin Jansa <martin.jansa@gmail.com> [120109 14:28]:
> 
>> On Mon, Jan 09, 2012 at 02:18:38PM +0100, Anders Darander wrote:
>>> Split systemd-analyze to a separate package, to remove the implicit 
>>> dependency on python. (At least the rpm-packaging backend created a 
>>> run-time dependency on python).
> 
>>> +PACKAGES =+ "${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs
>>> ${PN}-analyze"
> 
>>> USERADD_PACKAGES = "${PN}" GROUPADD_PARAM_${PN} = "-r lock"
> 
>>> +FILES_${PN}-analyze = "${bindir}/systemd-analyze"
> 
>> according to your commit message, why not add RDEPENDS_${PN}-analyze =
>> "python" here?
> 
> Yes, I should probably spin a v2 with: RDEPENDS_${PN}-analyze =
> "python-dbus" as python alone isn't enough to run systemd-analyze.

Don't forget python-pycairo for 'systemd-analyze plot'

regards,

Koen
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (Darwin)
Comment: GPGTools - http://gpgtools.org

iEYEARECAAYFAk8L6EMACgkQMkyGM64RGpFsMwCfYYRYu/3dMHcSatOubVTWgz8U
i8kAoI8iJsmGCdwTYWz492C5LJMn3uPH
=BrPL
-----END PGP SIGNATURE-----

Re: [PATCH 2/2] systemd: split out systemd-analyze

[Anders Darander]

* Koen Kooi <koen@dominion.thruhere.net> [120110 08:27]:

> Op 09-01-12 14:37, Anders Darander schreef:
> > * Martin Jansa <martin.jansa@gmail.com> [120109 14:28]:

> >> On Mon, Jan 09, 2012 at 02:18:38PM +0100, Anders Darander wrote:
> >>> Split systemd-analyze to a separate package, to remove the implicit 
> >>> dependency on python. (At least the rpm-packaging backend created a 
> >>> run-time dependency on python).

> >>> +PACKAGES =+ "${PN}-gui ${PN}-vconsole-setup ${PN}-initramfs
> >>> ${PN}-analyze"

> >>> USERADD_PACKAGES = "${PN}" GROUPADD_PARAM_${PN} = "-r lock"

> >>> +FILES_${PN}-analyze = "${bindir}/systemd-analyze"

> >> according to your commit message, why not add RDEPENDS_${PN}-analyze =
> >> "python" here?

> > Yes, I should probably spin a v2 with: RDEPENDS_${PN}-analyze =
> > "python-dbus" as python alone isn't enough to run systemd-analyze.

> Don't forget python-pycairo for 'systemd-analyze plot'

Thanks, I'll add it as a RRECOMMENDS and send an updated patch series.
(I've only used systemd an headless systems, without any graphics
libraries at all, thus never been running systemd-analyze plot)

I'll use RRECOMMENDS as it is not necessary for running systemd-analyze,
which python-dbus is. Well, it is necessary for running 'systemd-analyze
plot', but not for the other commands in systemd-analyze.

Cheers,
Anders

-- 
Anders Darander
ChargeStorm AB / eStorm AB