<html><body bgcolor="#FFFFFF"><div><span class="Apple-style-span" style="-webkit-tap-highlight-color: rgba(26, 26, 26, 0.296875); -webkit-composition-fill-color: rgba(175, 192, 227, 0.230469); -webkit-composition-frame-color: rgba(77, 128, 180, 0.230469); ">On 12.11.2010, at 09:48, Peter Stuge <<a href="mailto:peter@stuge.se">peter@stuge.se</a>> wrote:</span><br></div><div><br></div><div></div><blockquote type="cite"><div><blockquote type="cite"><blockquote type="cite"><span class="Apple-style-span" style="-webkit-tap-highlight-color: rgba(26, 26, 26, 0.296875); -webkit-composition-fill-color: rgba(175, 192, 227, 0.230469); -webkit-composition-frame-color: rgba(77, 128, 180, 0.230469); ">As soon as there are no more warnings I think it should be in the</span><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><span>commit hook.</span><br></blockquote></blockquote><blockquote type="cite"><span></span><br></blockquote><blockquote type="cite"><span>Not sure if it really belongs in the commit hook - the problem is</span><br></blockquote><blockquote type="cite"><span>that it blocks the commit, and already takes a while now. No need</span><br></blockquote><blockquote type="cite"><span>to make it even slower.</span><br></blockquote><span></span><br><span>It should block the commit, to stop ugly commits. Sure it takes a</span><br><span>while, but I think that's OK. QA is important. It depends on exactly</span><br><span>how long it will take..</span><font class="Apple-style-span" color="#000000"><font class="Apple-style-span" color="#0023A3"><br></font></font></div></blockquote><div><br></div>I think it should run asynchronously on commit just like abuild does. And add it's findings to the abuild mail report.<div><br></div><div>Or maybe abuild should call it?</div><div><br></div><div>Stefan<br><div><br></div><br><blockquote type="cite"><div><span></span><br><span>//Peter</span><br><span></span><br><span>-- </span><br><span>coreboot mailing list: <a href="mailto:coreboot@coreboot.org"><a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a></a></span><br><span><a href="http://www.coreboot.org/mailman/listinfo/coreboot">http://www.coreboot.org/mailman/listinfo/coreboot</a></span><br><span></span><br></div></blockquote></div></body></html>