Skip to content
New issue

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

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

Already on GitHub? # to your account

Include signal.h when compiling on various BSDs #203

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fabiankeil
Copy link

On ElectroBSD this fixes:

cc -DHAVE_CONFIG_H -I. -I../include -I/include/zlib -I/include -I/usr/local/include/openssl -I/usr/local/include -I/include/zlib -I/include -D_REENTRANT -D_POSIX_PTHREAD_SEMANTICS -W -Wall -Wunused-value -DOPENSSL_NO_KRB5 -g -O2 -c -o init.o init.c
init.c:78:47: error: use of undeclared identifier 'SIGINT'
if (WIFSIGNALED(ret) && (WTERMSIG(ret) == SIGINT || WTERMSIG(ret) == SIGQUIT))
^
init.c:78:74: error: use of undeclared identifier 'SIGQUIT'
if (WIFSIGNALED(ret) && (WTERMSIG(ret) == SIGINT || WTERMSIG(ret) == SIGQUIT))
^

The FreeBSD port benchmarks/siege has been using a similar
patch (different OS sorting) for several years now.

On ElectroBSD this fixes:

   cc -DHAVE_CONFIG_H -I. -I../include    -I/include/zlib -I/include -I/usr/local/include/openssl -I/usr/local/include -I/include/zlib -I/include -D_REENTRANT -D_POSIX_PTHREAD_SEMANTICS -W -Wall -Wunused-value -DOPENSSL_NO_KRB5  -g -O2 -c -o init.o init.c
   init.c:78:47: error: use of undeclared identifier 'SIGINT'
       if (WIFSIGNALED(ret) && (WTERMSIG(ret) == SIGINT || WTERMSIG(ret) == SIGQUIT))
                                                 ^
   init.c:78:74: error: use of undeclared identifier 'SIGQUIT'
       if (WIFSIGNALED(ret) && (WTERMSIG(ret) == SIGINT || WTERMSIG(ret) == SIGQUIT))
                                                                            ^

The FreeBSD port benchmarks/siege has been using a similar
patch (different OS sorting) for several years now.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant