Dear Richard,

Am Donnerstag, den 17.11.2011, 00:13 +0000 schrieb Richard Purdie:


unfortunately I could not find the meaning of upbreak here.

> We should be using libtool-cross for cross compiling but
> were not.

Is that just a cosmetic thing or did packages break?

> This patch sets datadir so libtoolize ends up
> containing correct paths. It then installs libtoolize.
> 
> The path ltmain.sh was installed to was incorrect and this is fixed.
> 
> We also now install all the libtool m4 macros.
> 
> Signed-off-by: Richard Purdie <richard.pur...@linuxfoundation.org>
> ---
> diff --git a/meta/recipes-devtools/libtool/libtool-cross_2.4.bb 
> b/meta/recipes-devtools/libtool/libtool-cross_2.4.bb
> index 596528a..4e6e3f2 100644
> --- a/meta/recipes-devtools/libtool/libtool-cross_2.4.bb
> +++ b/meta/recipes-devtools/libtool/libtool-cross_2.4.bb
> @@ -1,10 +1,13 @@
>  require libtool-${PV}.inc
>  
> -PR = "r5"
> +PR = "r7"

Increase just by one?

>  PACKAGES = ""
>  SRC_URI += "file://prefix.patch"
>  SRC_URI += "file://fixinstall.patch"
>  
> +target_datadir := "${datadir}"

Is not already defined this way in `bitbake.conf`?

$ git grep target_d ./meta/conf/bitbake.conf
meta/conf/bitbake.conf:target_datadir := "${datadir}"

> +datadir = "${STAGING_DIR_TARGET}${target_datadir}"
> +
>  do_configure_prepend () {
>       # Remove any existing libtool m4 since old stale versions would break
>       # any upgrade
> @@ -13,20 +16,21 @@ do_configure_prepend () {
>  }
>  
>  do_install () {
> -     install -d ${D}${bindir}/
> -     install -m 0755 ${HOST_SYS}-libtool ${D}${bindir}/${HOST_SYS}-libtool
> -     install -d ${D}${datadir}/libtool/
> -     install -d ${D}${datadir}/aclocal/
> -     install -c ${S}/libltdl/config/config.guess ${D}${datadir}/libtool/
> -     install -c ${S}/libltdl/config/config.sub ${D}${datadir}/libtool/
> -     install -c -m 0644 ${S}/libltdl/config/ltmain.sh ${D}${datadir}/libtool/
> -     install -c -m 0644 ${S}/libltdl/m4/libtool.m4 ${D}${datadir}/aclocal/
> -     install -c -m 0644 ${S}/libltdl/m4/ltdl.m4 ${D}${datadir}/aclocal/
> +     install -d ${D}${bindir_crossscripts}/
> +     install -m 0755 ${HOST_SYS}-libtool 
> ${D}${bindir_crossscripts}/${HOST_SYS}-libtool
> +     install -d ${D}${bindir_crossscripts}/
> +     install -m 0755 libtoolize ${D}${bindir_crossscripts}/
> +     install -d ${D}${target_datadir}/libtool/config/
> +     install -d ${D}${target_datadir}/aclocal/
> +     install -c ${S}/libltdl/config/config.guess 
> ${D}${target_datadir}/libtool/
> +     install -c ${S}/libltdl/config/config.sub ${D}${target_datadir}/libtool/
> +     install -c -m 0644 ${S}/libltdl/config/ltmain.sh 
> ${D}${target_datadir}/libtool/config/
> +     install -c -m 0644 ${S}/libltdl/m4/*.m4 ${D}${target_datadir}/aclocal/
>  }
>  
>  SYSROOT_PREPROCESS_FUNCS += "libtoolcross_sysroot_preprocess"
>  
>  libtoolcross_sysroot_preprocess () {
> -     install -d ${SYSROOT_DESTDIR}${bindir_crossscripts}
> -     install -m 755 ${D}${bindir}/${HOST_SYS}-libtool 
> ${SYSROOT_DESTDIR}${bindir_crossscripts}/${HOST_SYS}-libtool
> +     sysroot_stage_dir ${D}${bindir_crossscripts} 
> ${SYSROOT_DESTDIR}${bindir_crossscripts}
> +     sysroot_stage_dir ${D}${target_datadir} 
> ${SYSROOT_DESTDIR}${target_datadir}
>  }


Reviewed-by: Paul Menzel <paulepan...@users.sourceforge.net>


Thanks,

Paul

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to