fix compilation (partially) of freebsd v1 APPLIED

Lorenz (xha): 1
 fix compilation (partially) of freebsd

 1 files changed, 1 insertions(+), 0 deletions(-)

[PATCH] fix compilation (partially) of freebsd Export this patch

for freebsd, IPPROTO_TCP is defined in <netinet/in.h>. i didn't do an #ifedef
because this might be also the case for other platforms and it probably is a
good idea to include both <arpa/inet.h> and <netinet/in.h> ?!

Sebastian <sebastian@sebsite.pw>

1 year, 4 months ago

Yeah, arpa_inet.h(0p) says "Inclusion of the <arpa/inet.h> header may
also make visible all symbols from <netinet/in.h>". Both musl and glibc
do this; FreeBSD apparently doesn't. So the correct thing to do here is
to just unconditionally include <netinet/in.h>.

Thanks!

To gitsrht:~sebsite/generic-tetromino-game
   2ffcecf..73c5d99  main -> main

Next

---
also, for anyone looking to compile on freebsd, you'll need to install the
"pkgconf" and "lua54" packages and replace lua5.4 with lua-5.4 in the
makefile in order to build. maby a configure script would be a good idea? idk.

Sebastian <sebastian@sebsite.pw>

1 year, 4 months ago

That's pretty annoying :/ A configure script would work here but it
feels like overkill. I pushed a commit that adds a LUA macro to the
Makefile that defaults to lua5.4, so FreeBSD users will just need to run
`make LUA=lua-5.4`. I think pkgconf is a sensible dependency, though I
added this to the build instructions.

 main.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/main.c b/main.c
index d51d77c..6a2ebd8 100644
--- a/main.c
+++ b/main.c
@@ -10,6 +10,7 @@
 #include <sys/socket.h>
 
 #include <arpa/inet.h>
+#include <netinet/in.h>
 
 #include <assert.h>
 #include <netdb.h>
-- 
2.40.1