From: Thomas Rast <trast@student.ethz.ch>
Date: Sat, 24 Jul 2010 23:02:19 GMT
Subject: Re: [PATCH 3/6] Makefile: Split out the untested functions target
Message-ID: <201007250102.20099.trast@student.ethz.ch>
In-Reply-To: <1280004663-4887-4-git-send-email-avarab@gmail.com>

Ævar Arnfjörð Bjarmason wrote:
> Change the coverage-report target so that it doesn't generate the
> coverage-untested-functions file by default. I'm adding more targets
> for doing various things with the gcov files, and they shouldn't all
> run by default.
> 
> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
> ---
>  Makefile |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index 63f3f84..5e9a6a2 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2299,6 +2299,8 @@ coverage-report:
>  	gcov -b -o builtin builtin/*.c
>  	gcov -b -o xdiff xdiff/*.c
>  	gcov -b -o compat compat/*.c
> +
> +coverage-report-untested-functions:
>  	grep '^function.*called 0 ' *.c.gcov \
>  		| sed -e 's/\([^:]*\)\.gcov: *function \([^ ]*\) called.*/\1: \2/' \
>  		| tee coverage-untested-functions

This should depend on coverage-report, and either have its name
changed to coverage-untested-functions or be .PHONY.

-- 
Thomas Rast
trast@{inf,student}.ethz.ch