<html><head></head><body><div>On Thu, 2015-10-15 at 19:15 -0700, Christopher Larson wrote:</div><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 15, 2015 at 10:27 AM, Benjamin Esquivel <span dir="ltr">&lt;<a href="mailto:benjamin.esquivel@linux.intel.com" target="_blank">benjamin.esquivel@linux.intel.com</a>&gt;</span> wrote:<br><blockquote type="cite"><div id=":1fa" class="a3s" style="overflow:hidden">+&nbsp; &nbsp; def movefile(self, sourcefile, destdir):<br>
+&nbsp; &nbsp; &nbsp; &nbsp; try:<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; # FIXME: this check of movefile's return code to None should be<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; # fixed within the function to use only exceptions to signal when<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; # something goes wrong<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if (bb.utils.movefile(sourcefile, destdir) == None):<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; raise Exception("moving {} to {} failed".format(<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; sourcefile, destdir))<br>
+&nbsp; &nbsp; &nbsp; &nbsp; except Exception as e:<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; bb.warn(str(e))<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; bb.error("unable to place {} in final SDK location".format(<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; sourcefile))<br>
+<br>
+&nbsp; &nbsp; def mkdirhier(self, dirpath):<br>
+&nbsp; &nbsp; &nbsp; &nbsp; try:<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; bb.utils.mkdirhier(dirpath)<br>
+&nbsp; &nbsp; &nbsp; &nbsp; except OSError as e:<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; bb.warn(str(e))<br>
+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; bb.error("cannot make dir for SDK: {}".format(dirpath))</div><br></blockquote></div><br>Shouldn't this be fatal?<br></div></div></blockquote><div>correct, will change to fatal. Thanks.</div></body></html>