Re: [PATCH net] selftests: rds: move include.sh to TEST_FILES

[Hangbin Liu]

On Fri, Nov 22, 2024 at 03:01:29PM +0000, Maximilian Heyne wrote:
> Hi Hangbin,
> 
> On Fri, Sep 27, 2024 at 12:13:49PM +0800, Hangbin Liu wrote:
> > The include.sh file is generated for inclusion and should not be executable.
> > Otherwise, it will be added to kselftest-list.txt. Additionally, add the
> > executable bit for test.py at the same time to ensure proper functionality.
> > 
> > Fixes: 3ade6ce1255e ("selftests: rds: add testing infrastructure")
> > Signed-off-by: Hangbin Liu <liuhangbin@xxxxxxxxx>
> > ---
> >  tools/testing/selftests/net/rds/Makefile | 3 ++-
> >  tools/testing/selftests/net/rds/test.py  | 0
> >  2 files changed, 2 insertions(+), 1 deletion(-)
> >  mode change 100644 => 100755 tools/testing/selftests/net/rds/test.py
> > 
> > diff --git a/tools/testing/selftests/net/rds/Makefile b/tools/testing/selftests/net/rds/Makefile
> > index da9714bc7aad..cf30307a829b 100644
> > --- a/tools/testing/selftests/net/rds/Makefile
> > +++ b/tools/testing/selftests/net/rds/Makefile
> > @@ -4,9 +4,10 @@ all:
> >  	@echo mk_build_dir="$(shell pwd)" > include.sh
> >  
> >  TEST_PROGS := run.sh \
> > -	include.sh \
> >  	test.py
> 
> Should test.py also move down to TEST_FILES? I think run.sh is executing
> test.py anyway but does a couple of sanity checks before, so I think
> this it's not necessary to let the runner execute test.py standalone.

I was not aware of this. Thanks for the report. I will post a fix for this.

Thanks
Hangbin