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

[Emil Velikov]

On 19 March 2017 at 15:00, Eric Engestrom <eric at engestrom.ch> wrote:
> 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`?
>
That's correct - too much copy/pasta. Fixed locally.

-Emil