2010/10/18 Warren Turkal <span dir="ltr"><<a href="mailto:wt@penguintechs.org" target="_blank">wt@penguintechs.org</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div>On Sunday, October 17, 2010 09:37:27 am Idwer Vollering wrote:<br>
> Update support for FreeBSD.<br>
><br>
> Signed-off-by: Idwer Vollering <<a href="mailto:vidwer@gmail.com" target="_blank">vidwer@gmail.com</a>><br></div></blockquote><div><br>Updated patch attached. <br></div><div>Signed-off-by: Idwer Vollering <<a href="mailto:vidwer@gmail.com">vidwer@gmail.com</a>><br>
<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>
<br>
</div>A few questions:<br>
* Would it be better set LIBS to -lz and make LDFLAGS default to $(LIBS) on<br>
all platforms?<br>
* I thought that FreeBSD used GCC?<br></blockquote><div><br>It does, but you need GNU make (gmake): <a href="http://www.freebsd.org/cgi/url.cgi?ports/devel/gmake/pkg-descr" target="_blank">http://www.freebsd.org/cgi/url.cgi?ports/devel/gmake/pkg-descr</a> to be able to do a native compile (from trunk/ code -- from fbsd's ports is a different story).<br>

</div><div> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
 - If so, why remove -ansi and -pedantic on FreeBSD?<br></blockquote><div><br>This happens when leaving -pedantic (ergo not overriding  CFLAGS that's outside of the ifeq()..endif) alone:<br>- patch:<br><br>Index: Makefile<br>

===================================================================<br>--- Makefile    (revision 5965)<br>+++ Makefile    (working copy)<br>@@ -39,6 +39,11 @@<br> ifeq ($(OS_ARCH), Darwin)<br> LIBS = -framework IOKit -framework DirectIO -lpci -lz<br>

 endif<br>+ifeq ($(OS_ARCH), FreeBSD)<br>+CFLAGS +=-I/usr/local/include<br>+LDFLAGS += -L/usr/local/lib<br>+LIBS = -lz<br>+endif<br><br> # Support for PCI-attached "Super I/Os" (e.g. in VIA VT82686A/B).<br> CONFIG_PCI = yes<br>

<br>- error:<br><br>$ gmake clean ; gmake<br>rm -f superiotool *.o<br>gcc -O2 -Wall -Werror -Wstrict-prototypes -Wundef -Wstrict-aliasing -Werror-implicit-function-declaration -ansi -pedantic -D'SUPERIOTOOL_VERSION="5728"' -I/usr/local/include -DPCI_SUPPORT   -c -o superiotool.o superiotool.c<br>

cc1: warnings being treated as errors<br>In file included from superiotool.h:41,<br>                 from superiotool.c:24:<br>/usr/local/include/pci/pci.h:238: warning: comma at end of enumerator list<br>gmake: *** [superiotool.o] Error 1<br>

 <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
 - In either case, would it be better to replace -ansi with something like -<br>
std=c90 on all platforms and leave -pedantic? Would that be enough to work?<br></blockquote><div><br>cc1: error: unrecognized command line option "-std=c90"<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">


<br>
Thanks,<br>
<font color="#888888">wt<br>
</font></blockquote></div><br>