<div dir="ltr">Seems like this version was already picked to master-next, so probably better to leave as standalone change now.<div><br></div><div>The version currently in master fails to build for arm with thumb enabled:</div><div><a href="http://errors.yoctoproject.org/Errors/Details/155765/">http://errors.yoctoproject.org/Errors/Details/155765/</a><br></div><div><br></div><div>Is it fixed somewhere in this new patchset?</div><div><br></div><div><pre style="padding:9.5px;font-family:Monaco,Menlo,Consolas,"Courier New",monospace;font-size:13px;color:rgb(51,51,51);border-radius:4px;margin-top:0px;margin-bottom:10px;line-height:20px;word-break:break-all;word-wrap:break-word;white-space:pre-wrap;background-color:rgb(245,245,245);border:1px solid rgba(0,0,0,0.15)"># <a href="http://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">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</a>: 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)</pre></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 12, 2017 at 3:32 PM, Otavio Salvador <span dir="ltr"><<a href="mailto:otavio.salvador@ossystems.com.br" target="_blank">otavio.salvador@ossystems.com.br</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Sep 12, 2017 at 10:15 AM, Martin Jansa <<a href="mailto:martin.jansa@gmail.com">martin.jansa@gmail.com</a>> wrote:<br>
> Doesn't this belong into:<br>
> [OE-core] [PATCH v8 03/13] go: split out go-runtime into separate recipe<br>
> where it packaging change was probably introduced?<br>
<br>
</span>I can squash it; in fact the change was introduced on "[v8,05/13]<br>
go.bbclass: remove some xxx_FINAL variables". Not sure if it is better<br>
or worse in this case. What do you think?<br>
<span class=""><br>
> Also just RFC question, would it make sense to use:<br>
><br>
> VIRTUAL-RUNTIME_bash ?= "bash"<br>
> RDEPENDS_${PN}-dev += "${VIRTUAL-RUNTIME_bash}"<br>
><br>
> instead of the bash dependency directly? If it's acceptable for oe-core I'll<br>
> create change to do it for all recipe (I don't mind waiting after 2.4 is<br>
> released).<br>
><br>
> Because right now for reasons described in<br>
> <a href="https://bugzilla.yoctoproject.org/show_bug.cgi?id=9217" rel="noreferrer" target="_blank">https://bugzilla.yoctoproject.<wbr>org/show_bug.cgi?id=9217</a> I need to do the<br>
> above +<br>
> RDEPENDS_${PN}-dev_remove = "bash"<br>
> for many recipes with many bbappends, for our distro where we RPROVIDE bash<br>
> by busybox, which is good enough for package manager, but QA error is still<br>
> shown in cases where the real bash is built before the component rdpenending<br>
> on it.<br>
<br>
</span>I'd be fine with this. It does make sense to allow the replacement of bash.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Otavio Salvador O.S. Systems<br>
<a href="http://www.ossystems.com.br" rel="noreferrer" target="_blank">http://www.ossystems.com.br</a> <a href="http://code.ossystems.com.br" rel="noreferrer" target="_blank">http://code.ossystems.com.br</a><br>
Mobile: <a href="tel:%2B55%20%2853%29%209981-7854" value="+555399817854">+55 (53) 9981-7854</a> Mobile: <a href="tel:%2B1%20%28347%29%20903-9750" value="+13479039750">+1 (347) 903-9750</a><br>
</font></span></blockquote></div><br></div>