On 16.12.20 14:36, Johannes Berg wrote: > From: Johannes Berg <johannes.b...@intel.com> > > Perhaps something got moved around at some point, but the > current path that gets inserted has "/scripts/gdb" twice, > since the script is located in scripts/gdb/ already. Fix > the path. > > Signed-off-by: Johannes Berg <johannes.b...@intel.com> > --- > scripts/gdb/vmlinux-gdb.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/gdb/vmlinux-gdb.py b/scripts/gdb/vmlinux-gdb.py > index 4136dc2c59df..476da5b6a7ca 100644 > --- a/scripts/gdb/vmlinux-gdb.py > +++ b/scripts/gdb/vmlinux-gdb.py > @@ -13,7 +13,7 @@ > > import os > > -sys.path.insert(0, os.path.dirname(__file__) + "/scripts/gdb") > +sys.path.insert(0, os.path.dirname(__file__)) > > try: > gdb.parse_and_eval("0") >
How did you test that, which setup? I just ran "gdb <linux>/build/vmlinux", and "python print(sys.path)" didn't expose this duplication. So your change would actually break that common case. Jan -- Siemens AG, T RDA IOT Corporate Competence Center Embedded Linux