<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 13, 2014 at 9:54 AM, Burton, Ross <span dir="ltr"><<a href="mailto:ross.burton@intel.com" target="_blank">ross.burton@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb adM"><div class="h5">On 13 March 2014 16:46, Burton, Ross <<a href="mailto:ross.burton@intel.com">ross.burton@intel.com</a>> wrote:<br>
> On 11 March 2014 17:08, Robert Yang <<a href="mailto:liezhi.yang@windriver.com">liezhi.yang@windriver.com</a>> wrote:<br>
>> We need unset it after we use it, otherwise it would affect the env<br>
>> after we run "bb.build.exec_func('do_patch', d)", and will cause<br>
>> unexpected errors.<br>
><br>
> This assignment has been there since October 2012, so what are the<br>
> problems that this causes?<br>
<br>
</div></div>So what bothers me about this (and the original patch) is that TMPDIR<br>
has special meaning to bitbake and the conflict with patch's temporary<br>
directory is unfortunate. Maybe a better solution is to change<br>
meta/lib/oe/patch.py to set TMPDIR when invoking patch so the changed<br>
TMPDIR is constrained to the patch process and nothing else.</blockquote></div><br></div><div class="gmail_extra">That sounds like a clean approach to me.<br>-- <br>Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>
Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics
</div></div>