<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN">
<html><body style='font-family: Helvetica,Arial,sans-serif'>
<p>2016-02-23 12:36 skrev Markus Lehtonen:</p>
<blockquote type="cite" style="padding-left:5px; border-left:#1010ff 2px solid; margin-left:5px; width:100%"><!-- html ignored --><!-- head ignored --><!-- meta ignored -->
<pre>It would seem reasonable that do_unpack() always results in a pristine source
tree. This patch causes ${S} to be removed before unpacking sources.

I didn't quite understand what kind of "investigation" the following code
comment in do_unpack() was referring to:
  # TODO: Investigate if we can remove
  # the entire ${S} in this case.
What kind of special cases might we have? At least qemux86 core-image-sato
builds fine for me.<br /><br />

</pre>
</blockquote>
<p>The text isn't referring to any investigation already done. But me and Paul Eggleton discussed this matter (removing entire ${S}, not just {$S}/patches) within the scope of another bug. But since this wasn't really the scope for that bug, the TODO was added instead. And happily you have done the investigation and it's seems like ${S} might be able to be removed in this case, after all :)<br /><br />So please go ahead and you can remove that TODO comment within the scope of this patch, since if this change gets merged it's not relevant anymore..<br /><br />BR Petter</p>
<blockquote type="cite" style="padding-left:5px; border-left:#1010ff 2px solid; margin-left:5px; width:100%">[YOCTO #9064] Markus Lehtonen (1): base.bbclass wipe ${S} before unpacking source meta/classes/base.bbclass | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) <span class="sig">-- 2.6.2 </span></blockquote>
<p>See inline comments.</p>
<div>
<pre>Petter Mab&auml;cker

Technux &lt;petter@technux.se&gt;
www.technux.se</pre>
</div>
</body></html>