I think you're missing part of the patch, there should be a settings.h included, right?<br><br>-Corey<br><br><div class="gmail_quote">2009/4/1 Ronald Hoogenboom <span dir="ltr"><<a href="mailto:ronald@zonnet.nl">ronald@zonnet.nl</a>></span><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hi,<br>
<br>
I've attached a patch that removes the 3-mile-long compiler<br>
commandlines, which vim's quickfix doesn't like so much. Instead of<br>
putting all those -DXYZ='bla' on the compiler commandline, they are put<br>
in a file called settings.h (as #define XYZ bla) and only a<br>
--include=settings.h is put on the commandline.<br>
This file is created unconditionally at the same time as when the<br>
CPUFLAGS simply expanded make variable used to be created (not via a<br>
target rule and dependency), so it shouldn't change anything.<br>
<br>
Signed-off-by: Ronald Hoogenboom<<a href="mailto:hoogenboom30@zonnet.nl">hoogenboom30@zonnet.nl</a>><br>
<br>
<br>
<br>--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br></blockquote></div><br>