<div dir="ltr">On 1 November 2017 at 22:04, Richard Purdie <span dir="ltr">&lt;<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>&gt;</span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, 2017-10-31 at 21:25 -0200, Otavio Salvador wrote:<br>
&gt; On Tue, Oct 31, 2017 at 9:16 PM, Burton, Ross &lt;<a href="mailto:ross.burton@intel.com">ross.burton@intel.com</a>&gt;<br>
&gt; wrote:<br>
&gt; &gt;<br>
&gt; &gt; On 21 October 2017 at 13:06, Richard Purdie<br>
&gt; &gt; &lt;<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@<wbr>linuxfoundation.org</a>&gt; wrote:<br>
&gt; &gt; &gt; <br>
</span>&gt; &gt; &gt; diff --git a/scripts/oe-buildenv-internal b/scripts/oe-buildenv-<br>
<span class="">&gt; &gt; &gt; internal<br>
&gt; &gt; &gt; index e6cd141..abc5aff 100755<br>
&gt; &gt; &gt; --- a/scripts/oe-buildenv-internal<br>
&gt; &gt; &gt; +++ b/scripts/oe-buildenv-internal<br>
&gt; &gt; &gt; @@ -105,6 +105,11 @@ fi<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; Â BITBAKEDIR=$(readlink -f &quot;$BITBAKEDIR&quot;)<br>
&gt; &gt; &gt; Â BUILDDIR=$(readlink -f &quot;$BUILDDIR&quot;)<br>
&gt; &gt; &gt; +if [ -z &quot;$BBPATH&quot; ]; then<br>
&gt; &gt; &gt; +    BBPATH=$BUILDDIR<br>
&gt; &gt; &gt; +fi<br>
&gt; &gt; &gt; +<br>
&gt; &gt; &gt; +export BBPATH<br>
&gt; &gt;<br>
&gt; &gt; This breaks the use-case of the user switching between build<br>
&gt; &gt; directories as<br>
&gt; &gt; re-executing oe-init-build-env doesn&#39;t change BBPATH.<br>
&gt; I know and that&#39;s why I asked for a warning! ;-)<br>
<br>
</span>My patch as it stands clearly can&#39;t go in.<br>
<br>
Our options are:<br>
<br>
a) make that codepath unconditional (which may be no bad thing) and <br>
  Â use BBPATH<br>
b) use a specific variable for bitbake, the script sets BUILDDIR<br>
c) start allowing the external code to set TOPDIR directly<br>
<br>
I think I&#39;m leaning towards a) or possibly c). I don&#39;t really like b).<br>
I tried c) locally and its behaving oddly. a) is known to work.<br></blockquote><div><br></div><div>(a) seems like the right fix here.  At least, that&#39;s what I expected to happen...</div><div><br></div><div>Ross </div></div></div></div>