Re: [OE-core] [PATCH] guile: Remove bashisms

[He Zhe]

Thanks for pointing out.

This seems caused by the correction of installation path of guile-config(from 
${STAGING_BINDIR_CROSS} to ${SYSROOT_DESTDIR}${bindir_crossscripts}). We first 
put things in ${SYSROOT_DESTDIR}. And then do_populate_sysroot collects them 
from ${SYSROOT_DESTDIR} and install them into ${STAGING_BINDIR_CROSS}. But if 
we first put guile-configin ${STAGING_BINDIR_CROSS}, as we did before, and then 
apply this patch and then rebuild, the system will find guile-confighas existed 
in ${STAGING_BINDIR_CROSS} but cannot find who installed it(Matched in b''). So 
this is probably a wrong report. This error will not appear if I manually 
delete ${STAGING_BINDIR_CROSS}/guile-config and rebuild, or rebuild in a brand 
new project.

Zhe


On 10/10/2016 09:15 PM, Burton, Ross wrote:
>
> On 26 September 2016 at 07:51,  >wrote:
>
> Remove bashisms from do_populate_sysroot task
>
>
> This causes a change of behaviour that results in a stage error:
>
> ERROR: guile-2.0.12-r0 do_populate_sysroot: The recipe guile is trying to 
> install files into a shared area when those files already exist. Those files 
> and their manifest location are:
>
> /data/poky-master/tmp-glibc/sysroots/intel-corei7-64/usr/bin/crossscripts/guile-config
>  Matched in b''
> Please verify which recipe should provide the above files.
>
> Ross

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] guile: Remove bashisms

[Burton, Ross]

On 26 September 2016 at 07:51,  wrote:

> Remove bashisms from do_populate_sysroot task
>

This causes a change of behaviour that results in a stage error:

ERROR: guile-2.0.12-r0 do_populate_sysroot: The recipe guile is trying to
install files into a shared area when those files already exist. Those
files and their manifest location are:

 
/data/poky-master/tmp-glibc/sysroots/intel-corei7-64/usr/bin/crossscripts/guile-config
 Matched in b''
Please verify which recipe should provide the above files.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] guile: Remove bashisms

[He Zhe]

Hi Ross,

This one is the latest version for previously reviewed "[OE-core] [PATCH v2] 
Remove bashisms", but I forgot mentioning it.

Thanks,
Zhe

On 09/26/2016 02:51 PM, zhe...@windriver.com wrote:
> From: He Zhe 
>
> Remove bashisms from do_populate_sysroot task
>
> Signed-off-by: Peter Kjellerstedt 
> Signed-off-by: He Zhe 
> ---
>  meta/recipes-devtools/guile/guile_2.0.12.bb | 29 
> +
>  1 file changed, 17 insertions(+), 12 deletions(-)
>
> diff --git a/meta/recipes-devtools/guile/guile_2.0.12.bb 
> b/meta/recipes-devtools/guile/guile_2.0.12.bb
> index d2fe511..751a035 100644
> --- a/meta/recipes-devtools/guile/guile_2.0.12.bb
> +++ b/meta/recipes-devtools/guile/guile_2.0.12.bb
> @@ -87,22 +87,27 @@ SYSROOT_PREPROCESS_FUNCS = "guile_cross_config"
>  
>  guile_cross_config() {
>   # this is only for target recipe
> - if [ "${PN}" = "guile" ]
> - then
> - # Create guile-config returning target values instead of native 
> values
> - install -d ${SYSROOT_DESTDIR}${STAGING_BINDIR_CROSS}
> - echo '#!'`which ${BUILD_SYS}-guile`$' \\\n--no-auto-compile -e 
> main -s\n!#\n(define %guile-build-info '\'\( \
> - > ${B}/guile-config.cross
> - sed -n -e 's:^[ \t]*{[ \t]*":  (:' \
> + [ "${PN}" = "${BPN}" ] || return 0
> +
> + vars=$(sed -n -e 's:^[ \t]*{[ \t]*":  (:' \
>   -e 's:",[ \t]*": . ":' \
>   -e 's:" *}, *\\:"):' \
>   -e 's:^.*cachedir.*$::' \
>   -e '/^  (/p' \
> - < ${B}/libguile/libpath.h >> ${B}/guile-config.cross
> - echo '))' >> ${B}/guile-config.cross
> - cat ${B}/meta/guile-config >> ${B}/guile-config.cross
> - install ${B}/guile-config.cross 
> ${STAGING_BINDIR_CROSS}/guile-config
> - fi
> + < ${B}/libguile/libpath.h)
> +
> + # Create guile-config returning target values instead of native values
> + install -d ${SYSROOT_DESTDIR}${bindir_crossscripts}
> + cat <${B}/guile-config.cross
> +#!$(which ${BUILD_SYS}-guile) \\
> +--no-auto-compile -e main -s
> +!#
> +(define %guile-build-info '(
> +$vars
> +))
> +EOF
> + cat ${B}/meta/guile-config >> ${B}/guile-config.cross
> + install ${B}/guile-config.cross 
> ${SYSROOT_DESTDIR}${bindir_crossscripts}/guile-config
>  }
>  
>  # Guile needs the compiled files to be newer than the source, and it won't

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core