<br><br><div class="gmail_quote">On Fri, Jan 29, 2010 at 1:34 PM, Myles Watson <span dir="ltr"><<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br><br><div class="gmail_quote"><div class="im">On Fri, Jan 29, 2010 at 1:22 PM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de" target="_blank">stepan@coresystems.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">





  

<div><div>
On 1/29/10 9:16 PM, Myles Watson wrote:
<blockquote type="cite">
  
  

  
  <div>
  <p class="MsoNormal"><font size="2" color="navy" face="Arial"><span style="font-size: 10pt; font-family: Arial; color: navy;">> * don't
turn on WARNINGS_AS_ERRORS
for Qemu per default</span></font></p>
  
  
  <p class="MsoNormal"><font size="2" color="navy" face="Arial"><span style="font-size: 10pt; font-family: Arial; color: navy;">I disagree
with this change.  I think it
should be noticed when changes add warnings to a target that is free of
them. 
I think the goal should be to be able to turn this on for all targets. 
There
have been several times that warnings pointed out breakage.  Ignoring
them
seems counterproductive. <br></span></font></p>
  </div>
</blockquote></div>
Hm... It did not compile without warnings for me when I tried, so
that's why I disabled it at some point.</div></blockquote></div><div>It happens frequently, but the kbuildall report from 5063 says it built, and 5064 works for me. Hopefully tracking it will make it happen less often.  Each port that gets to the "warnings free" point will cover more and more of the code base.<br>

</div><div class="im"><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div text="#000000" bgcolor="#ffffff">

I generally agree with what you're saying.  Actually I manually deleted
the  "add -Werror-implicit-function-declaration to CFLAGS" part of my
patch because I figured people would feel no love for me if I left it
in. I think it should go in then, too.<br></div></blockquote></div><div>I guess that depends on how many boards it breaks and how long it would take us to fix them.<br></div></div></blockquote><div>* fix iasl output directory for i945 boards<br>
<br>I'd prefer it if the output went into $(obj)/mainboard/$(MAINBOARDDIR) instead of just $(obj).  I know it gets moved, but I still think it's nicer.<br><br>Minus the qemu change:<br>Acked-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
<br>Thanks,<br>Myles <br></div></div><br>