[pacman-dev] [PATCH 3/5] Add an include for signal.h when needed

Mark Weiman mark.weiman at markzz.com
Sat Apr 17 03:45:22 UTC 2021


On Linux, signal.h is not required to have access to the signal
constants. On FreeBSD, this is not the case and requires signal.h to be
explicitly included.

This patch adds an include for signal.h in any source file that uses it.

Signed-off-by: Mark Weiman <mark.weiman at markzz.com>
---
This is a modified patch that just adds signal.h to required files
instead of adding a check within meson.

 lib/libalpm/util.c | 1 +
 src/pacman/conf.c  | 1 +
 2 files changed, 2 insertions(+)

diff --git a/lib/libalpm/util.c b/lib/libalpm/util.c
index b386fde6..46c1d0a1 100644
--- a/lib/libalpm/util.c
+++ b/lib/libalpm/util.c
@@ -33,6 +33,7 @@
 #include <sys/socket.h>
 #include <fnmatch.h>
 #include <poll.h>
+#include <signal.h>
 
 /* libarchive */
 #include <archive.h>
diff --git a/src/pacman/conf.c b/src/pacman/conf.c
index a4f2ba35..cde96716 100644
--- a/src/pacman/conf.c
+++ b/src/pacman/conf.c
@@ -31,6 +31,7 @@
 #include <sys/utsname.h> /* uname */
 #include <sys/wait.h>
 #include <unistd.h>
+#include <signal.h>
 
 /* pacman */
 #include "conf.h"
-- 
2.31.1


More information about the pacman-dev mailing list