<div style="font-family: Helvetica; font-size: 13px; ">This series looks quite sane to me. Nicely done.<br></div>
<div><div><br></div><div>-- </div><div>Christopher Larson</div><div><br></div></div>
<p style="color: #A0A0A8;">On Wednesday, May 23, 2012 at 5:02 PM, Joshua Lock wrote:</p>
<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
<span><div><div><div>My recent change to the sanity class to warn users when they don't have R/W</div><div>permission to their SSTATE_CACHE directory has caused at least a couple of users</div><div>trouble, including the Yocto project autobuilder.</div><div><br></div><div>I have been unable to reproduce the issue but after discussionwith Elizabeth and</div><div>Chris on #yocto I came up with the following series.</div><div><br></div><div>The series includes a revert my original change as it seems the simple logic has</div><div>severla edge cases. I replace it later in the series with a simple piggy-back</div><div>on the existing check_create_long_filename() call, where I make the suggestion</div><div>of using SSTATE_MIRRORS if that call fails with "Permission denied" when called</div><div>against SSTATE_DIR. This check has been in use for some time and, to the best of</div><div>my knowledge, doesn't trigger invalid failures.</div><div><br></div><div>Thanks to Chris and Elizabeth for pointers as to why this was failing.</div><div><br></div><div>Cheers,</div><div><br></div><div>Joshua</div><div><br></div><div>The following changes since commit e6333825c3482a559a0c0499e17f8f48d3042ddf:</div><div><br></div><div> tune-mips64.inc: Add new tune file for mips64 big-endian (2012-05-20 20:24:37 -0700)</div><div><br></div><div>are available in the git repository at:</div><div> git://<a href="http://git.openembedded.org/openembedded-core-contrib">git.openembedded.org/openembedded-core-contrib</a> josh/sanity</div><div> <a href="http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=josh/sanity">http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=josh/sanity</a></div><div><br></div><div>Joshua Lock (7):</div><div> Revert "sanity.bbclass: check user can read and write to SSTATE_DIR"</div><div> sanity.bbclass: copy the data store and finalise before running</div><div> checks</div><div> sanity.bbclass: data.getVar(VAR, obj, exp) -> obj.getVar(VAR, exp)</div><div> sanity.bbclass: add newline to check_create_long_filename failure</div><div> message</div><div> sanity.bbclass: add extra information when SSTATE_CACHE unusable</div><div> sanity.bbclass: catch an extra exception in</div><div> check_create_long_filename</div><div> sanity.bbclass: check sanity at BuildStarted rather than ConfigParsed</div><div><br></div><div> meta/classes/sanity.bbclass | 153 ++++++++++++++++++++++---------------------</div><div> 1 files changed, 78 insertions(+), 75 deletions(-)</div><div><br></div><div>-- </div><div>1.7.7.6</div><div><br></div><div><br></div><div>_______________________________________________</div><div>Openembedded-core mailing list</div><div><a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a></div><div><a href="http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core">http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core</a></div></div></div></span>
</blockquote>
<div>
<br>
</div>