[oe] [PATCH] xfce_session_4.6.{1, 2}: explicitly pass the path of `iceauth`

[Khem Raj]

On (18/03/11 13:00), Paul Menzel wrote:
> Date: Fri, 18 Mar 2011 12:03:43 +0100
> 
> Xfce Session build depends on the executable `iceauth`, which fails when cross compiling for a different architecture.
> 
>         checking if the linker (arm-oe-linux-gnueabi-ld --sysroot=/oe/build-minimal-eglibc/minimal-dev/sysroots/armv5te-oe-linux-gnueabi) is GNU ld... (cached) yes
>         checking whether to build with profiling support... no
>         checking whether to compile with coverage profiling instrumentation... no
>         checking whether to disable assertions... no
>         checking whether to build final version... yes
>         checking whether arm-oe-linux-gnueabi-ld --sysroot=/oe/build-minimal-eglibc/minimal-dev/sysroots/armv5te-oe-linux-gnueabi accepts -O1... yes
>         checking for iceauth... no
>         configure: error: iceauth missing, please check your X11 installation
> 
> As suggested by Khem Raj [1] pass the value – `/usr/bin/iceauth` is taken – to the configure script so that the test is avoided.
> 
> Therefore `iceauth` is removed from `DEPENDS` and the `PR` variable is incremented.
> 
> This issue has been reported upstream as ticket #7420 [2].
> 
> [1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2011-March/031115.html
> [2] http://bugzilla.xfce.org/show_bug.cgi?id=7420
> 
> Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
> ---
>  recipes/xfce-base/xfce4-session_4.6.1.bb |    6 ++++--
>  recipes/xfce-base/xfce4-session_4.6.2.bb |    6 ++++--
>  2 files changed, 8 insertions(+), 4 deletions(-)
> 
> diff --git a/recipes/xfce-base/xfce4-session_4.6.1.bb b/recipes/xfce-base/xfce4-session_4.6.1.bb
> index ba66ac1..5029b21 100644
> --- a/recipes/xfce-base/xfce4-session_4.6.1.bb
> +++ b/recipes/xfce-base/xfce4-session_4.6.1.bb
> @@ -1,13 +1,15 @@
>  DESCRIPTION = "xfce4-session is a session manager for Xfce 4 Desktop Environment"
> -DEPENDS = "lbwnck libxfcegui4 libxfce4util dbus iceauth xfce-utils"
> +DEPENDS = "lbwnck libxfcegui4 libxfce4util dbus xfce-utils"
>  SECTION = "x11"
>  
> -PR = "r4"
> +PR = "r5"
>  
>  inherit xfce46
>  
>  RDEPENDS_${PN} = "iceauth xfce-utils xinit dbus-x11"
>  
> +EXTRA_OECONF += "ac_cv_path_ICEAUTH=/usr/bin/iceauth"

Would it make sense to have iceauth-native instead ?
> +
>  FILES_${PN} += "${libdir}/xfce4/splash/engines/*.so"
>  FILES_${PN} += "${datadir}/xfce4/tips/*"
>  FILES_${PN} += "${datadir}/themes/Default/balou/*"
> diff --git a/recipes/xfce-base/xfce4-session_4.6.2.bb b/recipes/xfce-base/xfce4-session_4.6.2.bb
> index d7f768a..c937775 100644
> --- a/recipes/xfce-base/xfce4-session_4.6.2.bb
> +++ b/recipes/xfce-base/xfce4-session_4.6.2.bb
> @@ -1,13 +1,15 @@
>  DESCRIPTION = "xfce4-session is a session manager for Xfce 4 Desktop Environment"
> -DEPENDS = "libwnck libxfcegui4 libxfce4util dbus iceauth xfce-utils"
> +DEPENDS = "libwnck libxfcegui4 libxfce4util dbus xfce-utils"
>  SECTION = "x11"
>  
> -PR = "r0"
> +PR = "r1"
>  
>  inherit xfce46
>  
>  RDEPENDS_${PN} = "iceauth xfce-utils xinit dbus-x11"
>  
> +EXTRA_OECONF += "ac_cv_path_ICEAUTH=/usr/bin/iceauth"
> +
>  FILES_${PN} += "${libdir}/xfce4/splash/engines/*.so"
>  FILES_${PN} += "${datadir}/xfce4/tips/*"
>  FILES_${PN} += "${datadir}/themes/Default/balou/*"
> -- 
> 1.7.4.1



> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel


-- 
-Khem