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

[Otavio Salvador]

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