From patchwork Thu Mar 21 08:24:28 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sven Eckelmann X-Patchwork-Id: 2794 Return-Path: Received: from v3-1039.vlinux.de (narfation.org [79.140.41.39]) by open-mesh.org (Postfix) with ESMTPS id 8E3E3601DDC for ; Thu, 21 Mar 2013 09:24:30 +0100 (CET) Received: from sven-desktop.localnet (drsd-4d05ed27.pool.mediaWays.net [77.5.237.39]) by v3-1039.vlinux.de (Postfix) with ESMTPSA id B6C43940D1; Thu, 21 Mar 2013 09:24:29 +0100 (CET) From: Sven Eckelmann To: Michael Tautschnig , 703540@bugs.debian.org Date: Thu, 21 Mar 2013 09:24:28 +0100 Message-ID: <3212285.GF4Btbehao@sven-desktop> User-Agent: KMail/4.8.4 (Linux/3.2.0-4-amd64; KDE/4.9.5; x86_64; ; ) In-Reply-To: <20130320175154.GT41481@l04.local> References: <20130320175154.GT41481@l04.local> MIME-Version: 1.0 Cc: b.a.t.m.a.n@lists.open-mesh.org Subject: Re: [B.A.T.M.A.N.] Bug#703540: Inconsistent use of _GNU_SOURCE X-BeenThere: b.a.t.m.a.n@lists.open-mesh.org X-Mailman-Version: 2.1.15 Precedence: list Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Mar 2013 08:24:30 -0000 tags 703540 + pending patch thanks On Wednesday 20 March 2013 17:51:54 you wrote: > While building the package using our research compiler infrastructure we > noticed conflicting types being used in the linked executable. This is due > to _GNU_SOURCE being defined in linux/route.c and posix/unix_socket.c, but > not in any other file. As a result, system headers expand to conflicting > declarations. (This was at least noticed for the sendto function, but may > extend to others.) > > Either all or no file should #define _GNU_SOURCE. Please add information how to reproduce this the next time you are adding such such a bug. Now I can just assume what you are writing is true (even when the man page about sendto says otherwise). Not knowing how to reproduce it in the best possible way just makes it harder for everyone to check the impact of the problem. I've forwarded it to the upstream maintainer and attached the change for Debian. Kind regards, Sven From 5041828ddb165b1c72c7fc4345aa6e0bec7abfa5 Mon Sep 17 00:00:00 2001 From: Sven Eckelmann Date: Thu, 21 Mar 2013 09:21:48 +0100 Subject: [PATCH] Define _GNU_SOURCE for all POSIX target source files Defining _GNU_SOURCE in source files differently can result in conflicting types. Signed-off-by: Sven Eckelmann --- debian/changelog | 3 +++ debian/patches/define_gnu_source.patch | 43 ++++++++++++++++++++++++++++++++ debian/patches/series | 1 + 3 files changed, 47 insertions(+) create mode 100644 debian/patches/define_gnu_source.patch create mode 100644 debian/patches/series diff --git a/debian/changelog b/debian/changelog index 31e69a6..5d08680 100644 --- a/debian/changelog +++ b/debian/changelog @@ -2,6 +2,9 @@ batmand (0.3.2+50+g06537ed-2) UNRELEASED; urgency=low * Upgraded to policy 3.9.4, no changes required * Remove obsolete DM-Upload-Allowed in debian/control + * debian/patches: + - Add define_gnu_source.patch, Define _GNU_SOURCE for all POSIX target + source files (Closes: #703540) -- Sven Eckelmann Wed, 19 Sep 2012 08:53:36 +0200 diff --git a/debian/patches/define_gnu_source.patch b/debian/patches/define_gnu_source.patch new file mode 100644 index 0000000..d81235f --- /dev/null +++ b/debian/patches/define_gnu_source.patch @@ -0,0 +1,43 @@ +Description: Define _GNU_SOURCE for all POSIX target source files + Defining _GNU_SOURCE in source files differently can result in conflicting + types. +Bug-Debian: #703540 +Author: Sven Eckelmann + +--- +diff --git a/Makefile b/Makefile +index fe1f094f310469331b3a7ff32e0eb2859bc5c998..ce1e198863aa311d5572e205f68534320d8f8431 100755 +--- a/Makefile ++++ b/Makefile +@@ -27,6 +27,7 @@ LINUX_OBJ = linux/route.o linux/tun.o linux/kernel.o + + ifeq ($(UNAME),Linux) + OS_OBJ = $(LINUX_OBJ) $(POSIX_OBJ) ++CPPFLAGS += -D_GNU_SOURCE + endif + + OBJ = batman.o originator.o schedule.o list-batman.o allocate.o bitarray.o hash.o profile.o ring_buffer.o hna.o $(OS_OBJ) +diff --git a/linux/route.c b/linux/route.c +index 119ebb2c08d0babfacdfa34ca38c74cc3160ff43..bceac95f1fc57c37bd18e1ada15d6f55582043a0 100644 +--- a/linux/route.c ++++ b/linux/route.c +@@ -21,7 +21,6 @@ + + + +-#define _GNU_SOURCE + #include + #include /* inet_ntop() */ + #include +diff --git a/posix/unix_socket.c b/posix/unix_socket.c +index 5bed217e286c247278046967783a197781c9a59e..02a0a3a97284fbf803565e12cebd8b22d305e66f 100644 +--- a/posix/unix_socket.c ++++ b/posix/unix_socket.c +@@ -21,7 +21,6 @@ + + + +-#define _GNU_SOURCE + #include + #include + #include diff --git a/debian/patches/series b/debian/patches/series new file mode 100644 index 0000000..f9b7991 --- /dev/null +++ b/debian/patches/series @@ -0,0 +1 @@ +define_gnu_source.patch -- 1.7.10.4