[B.A.T.M.A.N.] [PATCH] batmand: Don't override CPPFLAGS set by user

[B.A.T.M.A.N.] [PATCH] batmand: Don't override CPPFLAGS set by user

[Sven Eckelmann]

Users or build servers may want to enable preprocessor specific options like
-D_FORTIFY_SOURCE=2 that are usually placed in CPPFLAGS. The batmand makefile
should not override these options to add own preprocessor variables.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
---
 Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/Makefile b/Makefile
index 346b14a..65e0cf2 100755
--- a/Makefile
+++ b/Makefile
@@ -55,7 +55,7 @@ MANPAGE = man/batmand.8
 
 # batmand flags and options
 CFLAGS +=	-pedantic -Wall -W -std=gnu99 -MD
-CPPFLAGS =	-DDEBUG_MALLOC -DMEMORY_USAGE -DPROFILE_DATA $(NO_POLICY_ROUTING)
+CPPFLAGS +=	-DDEBUG_MALLOC -DMEMORY_USAGE -DPROFILE_DATA $(NO_POLICY_ROUTING)
 LDLIBS +=	-lpthread
 
 # disable verbose output
-- 
1.7.7.3

Re: [B.A.T.M.A.N.] [PATCH] batmand: Don't override CPPFLAGS set by user

[elektra]

> Users or build servers may want to enable preprocessor specific options
>  like -D_FORTIFY_SOURCE=2 that are usually placed in CPPFLAGS. The batmand
>  makefile should not override these options to add own preprocessor
>  variables.
> 
> Signed-off-by: Sven Eckelmann <sven@narfation.org>

Applied in 9cd7382.

Thanks!

Cheers,
Elektra

Re: [B.A.T.M.A.N.] [PATCH] batmand: Don't override CPPFLAGS set by user

[Sven Eckelmann]

[-- Attachment #1: Type: text/plain, Size: 840 bytes --]

On Tuesday 06 December 2011 20:53:53 elektra wrote:
> > Users or build servers may want to enable preprocessor specific options
> > 
> >  like -D_FORTIFY_SOURCE=2 that are usually placed in CPPFLAGS. The
> >  batmand makefile should not override these options to add own
> >  preprocessor variables.
> > 
> > Signed-off-by: Sven Eckelmann <sven@narfation.org>
> 
> Applied in 9cd7382.

It would be nice not to remove my authorship in git. It is enough to just save
the mail in a textfile (mbox) and run `git am patch123.mbox` or `git am -s
patch123.mbox` (this adds your Signed-off-by after my Signed-of-by). Now it
looks [1] like you are the author and committer of the patch.... but you
should only be the committer.

Kind regards,
	Sven

[1] http://git.open-mesh.org/?p=batmand.git;a=commitdiff;h=9cd7382f815fbebe25381d5ad5da4cd5074f20cc

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [B.A.T.M.A.N.] [PATCH] batmand: Don't override CPPFLAGS set by user

[elektra]

Hi -

thanks for the tip!
 
Cheers,
Elektra