Re: [PATCH 2/4] selftests: rds: add include.sh to EXTRA_CLEAN

Allison Henderson
2024-09-24T15:25:48Z

On Tue, 2024-09-24 at 14:49 +0200, Javier Carrasco wrote:
> The include.sh file is generated when building the net/rds selftests,
> but there is no rule to delete it with the clean target. Add the file
> to
> EXTRA_CLEAN in order to remove it when required.
>
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>

Ok, looks good. Thanks for catching this
Reviewed-by: Allison Henderson <allison.henderson@oracle.com>

> ---
>  tools/testing/selftests/net/rds/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/net/rds/Makefile
> b/tools/testing/selftests/net/rds/Makefile
> index da9714bc7aad..0b697669ea51 100644
> --- a/tools/testing/selftests/net/rds/Makefile
> +++ b/tools/testing/selftests/net/rds/Makefile
> @@ -7,6 +7,6 @@ TEST_PROGS := run.sh \
>         include.sh \
>         test.py
>
> -EXTRA_CLEAN := /tmp/rds_logs
> +EXTRA_CLEAN := /tmp/rds_logs include.sh
>
>  include ../../lib.mk
>