[Mesa-dev] [PATCH v2 01/23] glsl/tests: automake: cleanup all artefacts during clean-local

[Eric Engestrom]

On Friday, 2017-03-17 13:19:27 +0000, Emil Velikov wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
> 
> With later commits we'll fix the generators to produce the files in the
> correct location. That in itself will cause an issue since the files
> will be left dangling and make distcheck will fail.
> 
> Cc: Matt Turner <mattst88 at gmail.com>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
> Matt, do you think we should pick some of these in the tarball ?
> Alternatively we might want purge them all in the dist-hook.
> ---
>  src/compiler/Makefile.glsl.am | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/src/compiler/Makefile.glsl.am b/src/compiler/Makefile.glsl.am
> index f7be5ebce9d..66c82732584 100644
> --- a/src/compiler/Makefile.glsl.am
> +++ b/src/compiler/Makefile.glsl.am
> @@ -247,6 +247,12 @@ CLEANFILES +=						\
>  	glsl/glcpp/glcpp-lex.c
>  
>  clean-local:
> +	$(RM) -r glsl/tests/lower_jumps/*.opt_test
> +	$(RM) -r glsl/tests/lower_jumps/*.expected
> +	$(RM) -r glsl/tests/lower_jumps/*.out
> +	$(RM) -r glsl/tests/warnings/*.out
> +	$(RM) -r glsl/glcpp/tests/*.out
> +	$(RM) -r glsl/glcpp/tests/subtest*/

I think this line is the only one that needs the `-r`?

>  	$(RM) -r subtest-cr subtest-cr-lf subtest-lf subtest-lf-cr
>  
>  dist-hook:
> -- 
> 2.11.1
>