Re: [RFC][PATCH] WIP: feature-arm-thumb.inc: respect ARM_INSTRUCTION_SET when adding thumb suffix

[Martin Jansa <martin.jansa@gmail.com>]

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

On Tue, Jan 13, 2015 at 08:02:13PM +0100, Martin Jansa wrote:
> * this means that recipes with ARM_INSTRUCTION_SET explicitly changed
>   to arm will be built in feed without thumb suffix, the same does apply
>   for workdir, e.g. after "bitbake glib-2.0" you can see:
> 
>   tmp-glibc/work/armv5e-oe-linux-gnueabi:
>   glib-2.0  glibc  glibc-initial
> 
>   tmp-glibc/work/armv5te-oe-linux-gnueabi:
>   acl              db              gdk-pixbuf     kmod  ....
> 
>   and
> 
>   tmp-glibc/deploy/ipk:
>   all  armv5e  armv5te  qemuarm
> 
> * feed config should be ok, because all default DEFAULTTUNEs always
>   include "arm" variants of all supported PACKAGE_ARCHs
> 
> * for more details see
>   http://lists.openembedded.org/pipermail/openembedded-core/2014-April/091960.html
>   the toolchain path issues were resolved in 1.8

One more issue is left.

ERROR: Logfile of failure stored in: /OE/build/oe-core/tmp-glibc/work/armv5te-oe-linux-gnueabi/glibc-locale/2.20-r0/temp/log.do_install.21696
Log data follows:
| DEBUG: Executing shell function do_install
| ls: cannot access /OE/build/oe-core/tmp-glibc/sysroots/qemuarm/usr/include/glibc-locale-internal-armv5te-oe-linux-gnueabi//usr/bin: No such file or directory
| ls: cannot access /OE/build/oe-core/tmp-glibc/sysroots/qemuarm/usr/include/glibc-locale-internal-armv5te-oe-linux-gnueabi//usr/lib/locale: No such file or directory
| cp: cannot stat '/OE/build/oe-core/tmp-glibc/sysroots/qemuarm/usr/include/glibc-locale-internal-armv5te-oe-linux-gnueabi/SUPPORTED': No such file or directory
| WARNING: /OE/build/oe-core/tmp-glibc/work/armv5te-oe-linux-gnueabi/glibc-locale/2.20-r0/temp/run.do_install.21696:1 exit 1 from
|   cp -fpPR /OE/build/oe-core/tmp-glibc/sysroots/qemuarm/usr/include/glibc-locale-internal-armv5te-oe-linux-gnueabi/SUPPORTED /OE/build/oe-core/tmp-glibc/work/armv5te-oe-linux-gnueabi/glibc-locale/2.20-r0
| ERROR: Function failed: do_install (log file is located at /OE/build/oe-core/tmp-glibc/work/armv5te-oe-linux-gnueabi/glibc-locale/2.20-r0/temp/log.do_install.21696)
ERROR: Task 297 (/OE/build/oe-core/openembedded-core/meta/recipes-core/glibc/glibc-locale_2.20.bb, do_install) failed with exit code '1'

glibc-locale and glibc-scripts have PKGARCH in path, but because glibc is "arm" it's actually installed in armv5e (not armv5te):

/OE/build/oe-core/tmp-glibc/sysroots/qemuarm/usr/include/glibc-locale-internal-armv5e-oe-linux-gnueabi/
/OE/build/oe-core/tmp-glibc/sysroots/qemuarm/usr/include/glibc-scripts-internal-armv5e-oe-linux-gnueabi/

> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  meta/conf/machine/include/arm/feature-arm-thumb.inc | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/conf/machine/include/arm/feature-arm-thumb.inc b/meta/conf/machine/include/arm/feature-arm-thumb.inc
> index 8e6619d..4c4eb6d 100644
> --- a/meta/conf/machine/include/arm/feature-arm-thumb.inc
> +++ b/meta/conf/machine/include/arm/feature-arm-thumb.inc
> @@ -32,7 +32,8 @@ python () {
>  TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'thumb', ' -m${ARM_M_OPT}', '', d)}"
>  OVERRIDES .= "${@bb.utils.contains('TUNE_FEATURES', 'thumb', ':thumb', '', d)}"
>  
> -ARMPKGSFX_THUMB .= "${@bb.utils.contains('TUNE_FEATURES', 'thumb', '${ARM_THUMB_SUFFIX}', '', d)}"
> +# Add suffix from ARM_THUMB_SUFFIX only if after all this we still set ARM_M_OPT to thumb
> +ARMPKGSFX_THUMB .= "${@ d.getVar('ARM_THUMB_SUFFIX', True) if d.getVar('ARM_M_OPT', True) == 'thumb' else ''}"
>  
>  # Whether to compile with code to allow interworking between the two
>  # instruction sets. This allows thumb code to be executed on a primarily
> -- 
> 2.2.1
> 

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]