From 6eab073c07bb0880934db38933bf934593d29559 Mon Sep 17 00:00:00 2001 From: Andrey Vagin Date: Tue, 19 Aug 2014 15:49:00 +0400 Subject: [PATCH] zdtm: fix compile errors of zdtm_ct MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When "make test" is executed, CFLAGS is exported from the root Makefile. These flags define _GNU_SOURCE, so we don't need to define it in the souce file. In addition unistd.h isn't included, so a few functions are shown as undeclared. make zdtm_ns make[3]: Entering directory `/root/criu/test' gcc -O2 -Wall -Werror -DCONFIG_X86_64 -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE zdtm_ct.c -o zdtm_ct zdtm_ct.c:1:0: error: "_GNU_SOURCE" redefined [-Werror] #define _GNU_SOURCE ^ :0:0: note: this is the location of the previous definition zdtm_ct.c: In function ‘main’: zdtm_ct.c:21:2: error: implicit declaration of function ‘fork’ [-Werror=implicit-function-declaration] pid = fork(); ^ zdtm_ct.c:23:3: error: implicit declaration of function ‘setsid’ [-Werror=implicit-function-declaration] if (setsid() == -1) { ^ zdtm_ct.c:49:3: error: implicit declaration of function ‘execv’ [-Werror=implicit-function-declaration] execv(argv[1], argv + 1); ^ zdtm_ct.c:62:3: error: implicit declaration of function ‘getpid’ [-Werror=implicit-function-declaration] kill(getpid(), WTERMSIG(status)); ^ cc1: all warnings being treated as errors Signed-off-by: Andrey Vagin Tested-by: Ruslan Kuprieiev Signed-off-by: Pavel Emelyanov --- test/Makefile | 1 + test/zdtm_ct.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/test/Makefile b/test/Makefile index 9dcc52336..ed8c34bf8 100644 --- a/test/Makefile +++ b/test/Makefile @@ -28,6 +28,7 @@ fault-injection: .FORCE zdtm_ns: $(shell echo "$(TST)" | tr ' ' '\n' | grep -P $(EXP)) zdtm_nons: $(shell echo "$(TST)" | tr ' ' '\n' | grep -vP $(EXP)) +override CFLAGS += -D_GNU_SOURCE zdtm_ct: zdtm_ct.c clean: diff --git a/test/zdtm_ct.c b/test/zdtm_ct.c index 1374f786f..3471655fd 100644 --- a/test/zdtm_ct.c +++ b/test/zdtm_ct.c @@ -1,10 +1,10 @@ -#define _GNU_SOURCE #include #include #include #include #include #include +#include int main(int argc, char **argv) {