From d9996e6e934b4537f30a300889303d0406d5d9b3 Mon Sep 17 00:00:00 2001 From: Christophe CURIS Date: Tue, 20 Jan 2015 22:04:11 +0100 Subject: [PATCH] make: moved PHONY outside the USE_LCOV conditional Apparently, automake does not like when there are multiple .PHONY definitions and some of them are conditionals, it generates a warning: Makefile.am:84: warning: .PHONY was already defined in condition USE_LCOV, which is included in condition TRUE ... Makefile.am:58: ... '.PHONY' previously defined here This is probably a false message, because in the end, conditional or not, the result will not be significantly different, so this patch just moves the .PHONY definition outside the conditional so Automake will not print the warning. Signed-off-by: Christophe CURIS --- Makefile.am | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index 55667a1a..2357d242 100644 --- a/Makefile.am +++ b/Makefile.am @@ -44,6 +44,9 @@ EXTRA_DIST = TODO BUGS BUGFORM FAQ INSTALL \ script/generate-txt-from-texi.sh \ script/nested-func-to-macro.sh + +.PHONY: coverage-reset coverage + if USE_LCOV coverage-reset: find . -type f -name '*.gcda' -exec rm -f '{}' ';' @@ -54,8 +57,6 @@ coverage: -mkdir -p coverage @lcov_output_directory@ lcov --compat-libtool --capture --directory . --output-file coverage/coverage.info genhtml --output-directory @lcov_output_directory@ coverage/coverage.info - -.PHONY: coverage-reset coverage endif # make update-lang PO=