diff options
author | Michael Forney | 2021-01-13 22:56:00 +0100 |
---|---|---|
committer | Thomas Huth | 2021-01-20 10:42:03 +0100 |
commit | c0cb758eec848dcbc779bf75a26b49727218e380 (patch) | |
tree | 4a0e5cee7e1531c936bef4f078aa7bd2571708b2 /include/qemu | |
parent | Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20210119-... (diff) | |
download | qemu-c0cb758eec848dcbc779bf75a26b49727218e380.tar.gz qemu-c0cb758eec848dcbc779bf75a26b49727218e380.tar.xz 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 'include/qemu')
-rw-r--r-- | include/qemu/osdep.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/include/qemu/osdep.h b/include/qemu/osdep.h index f9ec8c84e9..a434382c58 100644 --- a/include/qemu/osdep.h +++ b/include/qemu/osdep.h @@ -104,10 +104,6 @@ extern int daemon(int, int); #include <setjmp.h> #include <signal.h> -#ifdef HAVE_SYS_SIGNAL_H -#include <sys/signal.h> -#endif - #ifndef _WIN32 #include <sys/wait.h> #else |