Re: [PATCH v3] xinit: Fix `startx` looking for `mcookie` in sysroot

From: Saul Wold <sgw@linux.intel.com>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Cc: Paul Menzel <paulepanter@users.sourceforge.net>
Subject: Re: [PATCH v3] xinit: Fix `startx` looking for `mcookie` in sysroot
Date: Thu, 10 Nov 2011 09:20:27 -0800	[thread overview]
Message-ID: <4EBC07DB.2090907@linux.intel.com> (raw)
In-Reply-To: <1320829161.4622.2.camel@mattotaupa>

On 11/09/2011 12:59 AM, Paul Menzel wrote:
> Date: Tue, 8 Nov 2011 13:00:32 +0100
>
> `startx` run on a system based on the demo systemd image [1] and `opkg`-installed packages fails with the following error.
>
> 	/usr/bin/startx: line 139: /OE/tentacle/build/tmp-angstrom_2010_x/sysroots/x68_64-linux/usr/bin/mcookie: No such file or directory
>
> Applying commit 443bcc07 [1] from OE-classic
>
>          Author: Tom Rini<tom_rini@mentor.com>
>          Date:   Thu Apr 7 10:36:43 2011 -0700
>
>              xinit: Fix mcookie / util-linux-ng dependency
>
>              xinit just needs to know the runtime path of mcookie so we need to
>              RDEPEND on util-linux-ng and pass the runtime path in via EXTRA_OECONF
>
>              Signed-off-by: Tom Rini<tom_rini@mentor.com>
>
> fixes this issue. Commit 7f6cec6f [2]
>
>          Author: Frans Meulenbroeks<fransmeulenbroeks@gmail.com>
>          Date:   Sun Feb 21 18:11:30 2010 +0100
>
>              xinit: add dependency on util-linux-ng
>
>          […]
>
> tried to address the same problem but apparently did not help, because Tom still had problems.
>
> [1] http://www.angstrom-distribution.org/demo/beagleboard/Angstrom-systemd-image-eglibc-ipk-v2011.11-core-beagleboard.rootfs.tar.bz2
> [2] http://git.openembedded.org/openembedded/commit/443bcc0785bc004e471b3750a34d12d2fd2e5dad
> [3] http://git.openembedded.org/openembedded/commit/7f6cec6f0adb6203a6dbaf8a43c67c2c4f8bf84e
>
> Signed-off-by: Paul Menzel<paulepanter@users.sourceforge.net>
> ---
> 1. Please note that I did neither build nor run tested this patch and
> just noticed the error trying the demo image.
> 2. Koen mentioned something on IRC wanting to get rid of `startx`
> entirely. But this patch should be applied nevertheless.
>
> v2: move `RDEPENDS` further down as suggested by Koen
> v3: there is no util-linux-ng in OE-core as noted by Saul
> ---
>   meta/recipes-graphics/xorg-app/xinit_1.3.0.bb |    7 ++++++-
>   1 files changed, 6 insertions(+), 1 deletions(-)
>
> diff --git a/meta/recipes-graphics/xorg-app/xinit_1.3.0.bb b/meta/recipes-graphics/xorg-app/xinit_1.3.0.bb
> index ee7c64d..6953ccc 100644
> --- a/meta/recipes-graphics/xorg-app/xinit_1.3.0.bb
> +++ b/meta/recipes-graphics/xorg-app/xinit_1.3.0.bb
> @@ -9,9 +9,14 @@ systems. When this first client exits, xinit will kill the X server and \
>   then terminate."
>
>   LIC_FILES_CHKSUM = "file://COPYING;md5=0d4b5eef75f1584ccbdc5e4a34314407"
> -PR = "r0"
> +
> +PR = "r1"
>   PE = "1"
>
> +EXTRA_OECONF = "ac_cv_path_MCOOKIE=${bindir}/mcookie"
> +
> +RDEPENDS_${PN} += "util-linux"
> +
>   FILES_${PN} += "${libdir}X11/xinit"
>
>   SRC_URI[md5sum] = "bc4e8b7d1919597cc37a0d24aa149dda"
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Merged into OE-Core

Thanks
	Sau!