Subject: pkg/35357: [patch] net/rtorrent does not build on Solaris
To: None <firstname.lastname@example.org, email@example.com,>
From: None <firstname.lastname@example.org>
Date: 01/04/2007 02:30:00
>Synopsis: [patch] net/rtorrent does not build on Solaris
>Arrival-Date: Thu Jan 04 02:30:00 +0000 2007
>Originator: Peter Schuller
>Release: Solaris nvb54
SunOS hostname 5.11 snv_54 i86pc i386 i86pc
The following issues cause build problems on solaris:
* Solaris does not have AF_LOCAL, only AF_UNIX.
* signal_handler.h includes <sys/signal.h> instead of <signal.h>. The result is that signal() is not declared. I have not actually bothered to check exactly why this happens to be so on Solaris, but given that sys/signal.h is seemingly not part of any relevant standard, and the fact that the manpage on Solaris, FreeBSD and NetBSD and Linux all say <signal.h> is the official manpage to include, I presume the inclusion of <sys/signal.h> is incorrect.
This patch fixes these issues:
It patches rak/socket_address.h to use AF_UNIX instead of AF_LOCAL for the af_local constant.
It also patches signal_handler.h to to unconditionally (regardless of platform) include <signal.h> instead of <sys/signal.h>.
Tested to still build on FreeBSD aswell.
(Note: This makes it build. There is at least one runtime issue, possibly having to do with curses, that I have not examined. Will file a separate PR if/when I look at it.)