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

Compilation error on FreeBSD 8.1 amd64 #9

Closed
gblach opened this issue Nov 21, 2010 · 1 comment
Closed

Compilation error on FreeBSD 8.1 amd64 #9

gblach opened this issue Nov 21, 2010 · 1 comment

Comments

@gblach
Copy link

gblach commented Nov 21, 2010

gcc on FreeBSD don't known how to use 'signal' subroutine without signal.h included in source.

Patch bellow:

---- BEGIN ----
Files antirez-hiredis-3be1d6e.orig/hiredis-test and antirez-hiredis-3be1d6e/hiredis-test differ
diff -ruN antirez-hiredis-3be1d6e.orig/test.c antirez-hiredis-3be1d6e/test.c
--- antirez-hiredis-3be1d6e.orig/test.c 2010-11-03 11:50:46.000000000 +0100
+++ antirez-hiredis-3be1d6e/test.c 2010-11-21 17:06:00.257564584 +0100
@@ -5,6 +5,9 @@
#include <sys/time.h>
#include <assert.h>
#include <unistd.h>
+#ifdef FreeBSD
+#include <signal.h>
+#endif

#include "hiredis.h"

Files antirez-hiredis-3be1d6e.orig/test.o and antirez-hiredis-3be1d6e/test.o differ
---- END ----

@pietern
Copy link
Contributor

pietern commented Nov 22, 2010

This has already been fixed in 1c39483. The conditional for FreeBSD is not required as the include should be there for every OS. Thanks for the report.

This issue was closed.
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