diff options
| author | Michael Forney <mforney@mforney.org> | 2021-01-13 13:56:00 -0800 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2021-01-20 10:42:03 +0100 |
| commit | c0cb758eec848dcbc779bf75a26b49727218e380 (patch) | |
| tree | 4a0e5cee7e1531c936bef4f078aa7bd2571708b2 /subprojects/libvhost-user/libvhost-user.h | |
| parent | 48202c712412c803ddb56365c7bca322aa4e7506 (diff) | |
| download | focaccia-qemu-c0cb758eec848dcbc779bf75a26b49727218e380.tar.gz focaccia-qemu-c0cb758eec848dcbc779bf75a26b49727218e380.zip | |
osdep.h: Remove <sys/signal.h> include
Prior to 2a4b472c3c, sys/signal.h was only included on OpenBSD
(apart from two .c files). The POSIX standard location for this
header is just <signal.h> and in fact, OpenBSD's signal.h includes
sys/signal.h itself.
Unconditionally including <sys/signal.h> on musl causes warnings
for just about every source file:
/usr/include/sys/signal.h:1:2: warning: #warning redirecting incorrect #include <sys/signal.h> to <signal.h> [-Wcpp]
1 | #warning redirecting incorrect #include <sys/signal.h> to <signal.h>
| ^~~~~~~
Since there don't seem to be any platforms which require including
<sys/signal.h> in addition to <signal.h>, and some platforms like
Haiku lack it completely, just remove it.
Tested building on OpenBSD after removing this include.
Signed-off-by: Michael Forney <mforney@mforney.org>
Tested-by: John Snow <jsnow@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20210113215600.16100-1-mforney@mforney.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'subprojects/libvhost-user/libvhost-user.h')
0 files changed, 0 insertions, 0 deletions