[OE-core] [PATCH v8 06/13] go-dep: Move bash dependency to -dev package

[Martin Jansa]

Seems like this version was already picked to master-next, so probably
better to leave as standalone change now.

The version currently in master fails to build for arm with thumb enabled:
http://errors.yoctoproject.org/Errors/Details/155765/

Is it fixed somewhere in this new patchset?

# github.com/golang/dep/cmd/dep
TOPDIR/tmp-glibc/work/armv5te-oe-linux-gnueabi/go-dep/0.3.0-r0/recipe-sysroot-native/usr/lib/arm-oe-linux-gnueabi/go/pkg/tool/linux_amd64/link:
R_ARM_THM_CALL, are you using -marm?
WARNING: exit code 2 from a shell command.
ERROR: Function failed: do_compile (log file is located at
TOPDIR/tmp-glibc/work/armv5te-oe-linux-gnueabi/go-dep/0.3.0-r0/temp/log.do_compile.13531)


On Tue, Sep 12, 2017 at 3:32 PM, Otavio Salvador <
otavio.salvador at ossystems.com.br> wrote:

> On Tue, Sep 12, 2017 at 10:15 AM, Martin Jansa <martin.jansa at gmail.com>
> wrote:
> > Doesn't this belong into:
> > [OE-core] [PATCH v8 03/13] go: split out go-runtime into separate recipe
> > where it packaging change was probably introduced?
>
> I can squash it; in fact the change was introduced on "[v8,05/13]
> go.bbclass: remove some xxx_FINAL variables". Not sure if it is better
> or worse in this case. What do you think?
>
> > Also just RFC question, would it make sense to use:
> >
> > VIRTUAL-RUNTIME_bash ?= "bash"
> > RDEPENDS_${PN}-dev += "${VIRTUAL-RUNTIME_bash}"
> >
> > instead of the bash dependency directly? If it's acceptable for oe-core
> I'll
> > create change to do it for all recipe (I don't mind waiting after 2.4 is
> > released).
> >
> > Because right now for reasons described in
> > https://bugzilla.yoctoproject.org/show_bug.cgi?id=9217 I need to do the
> > above +
> > RDEPENDS_${PN}-dev_remove = "bash"
> > for many recipes with many bbappends, for our distro where we RPROVIDE
> bash
> > by busybox, which is good enough for package manager, but QA error is
> still
> > shown in cases where the real bash is built before the component
> rdpenending
> > on it.
>
> I'd be fine with this. It does make sense to allow the replacement of bash.
>
> --
> Otavio Salvador                             O.S. Systems
> http://www.ossystems.com.br        http://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20170912/2a4eee87/attachment-0002.html>