From: Simon Marchi Date: Mon, 3 Feb 2020 22:38:23 +0000 (-0500) Subject: tests: append to AM_CFLAGS instead of overriding it X-Git-Tag: v2.13.0-rc1~754 X-Git-Url: http://git.liburcu.org/?a=commitdiff_plain;h=2868dcc95f37ab202cea1e4a3a00a88f7b42ba86;hp=2868dcc95f37ab202cea1e4a3a00a88f7b42ba86;p=lttng-tools.git tests: append to AM_CFLAGS instead of overriding it The Makefiles modified by this patch currently override the AM_CFLAGS value, which means that anything put in AM_CFLAGS by configure (for example, the warning flags) is lost. I believe the intention is to add some flags to CFLAGS, so modify them such that they append instead of override. notification/Makefile.am overrides AM_LDFLAGS with nothing. It feels like it was not the intention to actually clear that variable, but rather that it is just the by-product of a copy paste. If it was really the intention to clear the value of AM_LDFLAGS, there would have been a comment to explain it, right? Signed-off-by: Simon Marchi Change-Id: I4ef926d9135b16200e5f17d09461506a5e955068 Signed-off-by: Jérémie Galarneau ---