Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build on FreeBSD 8.4 fails, sequence of #includes in src/networking.c (with patch) #100

Closed
opsec opened this issue Jan 14, 2015 · 6 comments

Comments

@opsec
Copy link

opsec commented Jan 14, 2015

--- src/networking.c.orig 2015-01-14 05:12:08 UTC
+++ src/networking.c
@@ -1,9 +1,9 @@
#include <arpa/inet.h>
#include <fcntl.h>
-#include <netinet/in.h>
-#include <netinet/tcp.h>
#include <sys/socket.h>
#include <sys/types.h>
+#include <netinet/in.h>
+#include <netinet/tcp.h>
#include <sys/uio.h>
#include <syslog.h>
#include <unistd.h>

fixes it.

@opsec
Copy link
Author

opsec commented Jan 14, 2015

@armon
Copy link
Collaborator

armon commented Jan 14, 2015

@opsec Could you clarify the patch? It looks like it was only partially included.

@opsec
Copy link
Author

opsec commented Jan 14, 2015

Ups, sorry, cut and paste with the textfield. You can find it here:
https://lists.freebsd.org/pipermail/svn-ports-all/2015-January/083096.html

I'm not yet familiar with the process to submit the patch directly to github (sorry).

@armon
Copy link
Collaborator

armon commented Jan 14, 2015

@opsec That patch is just moving the import lower? Is that actually affecting things? I'm happy to move the import if it matters but it seems like it wouldn't.

@opsec
Copy link
Author

opsec commented Jan 14, 2015

Yes, it moves two includes below sys/types.h etc. Yes, it fixes the build on FreeBSD 8.4, where src/networking.c will not build otherwise.

@armon armon closed this as completed in c41dbbc Jan 14, 2015
@armon
Copy link
Collaborator

armon commented Jan 14, 2015

Cool, should be fixed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants