On 24/10/09 07:40, Jakub Kicinski wrote: > On Tue, 8 Oct 2024 10:22:53 +0200 Alessandro Zanni wrote: > > This fix solves this error, when calling kselftest with targets "net/rds": > > > > selftests: net/rds: test.py > > Traceback (most recent call last): > > File "tools/testing/selftests/net/rds/./test.py", line 17, in <module> > > from lib.py import ip > > ModuleNotFoundError: No module named 'lib' > > > > Signed-off-by: Alessandro Zanni <alessandro.zann...@gmail.com> > > --- > > tools/testing/selftests/net/rds/test.py | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/tools/testing/selftests/net/rds/test.py > > b/tools/testing/selftests/net/rds/test.py > > index e6bb109bcead..112a8059c030 100755 > > --- a/tools/testing/selftests/net/rds/test.py > > +++ b/tools/testing/selftests/net/rds/test.py > > @@ -14,8 +14,9 @@ import sys > > import atexit > > from pwd import getpwuid > > from os import stat > > -from lib.py import ip > > > > +sys.path.append("..") > > +from lib.py.utils import ip > > > > libc = ctypes.cdll.LoadLibrary('libc.so.6') > > setns = libc.setns > > Does this work regardless of where we try to run the script from? > In other cross-imports we try to build the path based on __file__, > see: tools/testing/selftests/drivers/net/lib/py/__init__.py
Yes, it works regardeless where we run the script from but I agree with you to keep the same style used in other files. Thanks for pointing out. > Would be good to keep consistency. Definitely. I'm sending a v2 patch with this in mind. > -- > pw-bot: cr