remove sys.path manipulation around builder.string_build (closes #76159)

That has probably been added in the past as a work around to have relative
import working. Not sure if it's still necessary or not, but that's not
a correct solution anyway, as this may cause pb with some module wrongly
named and added to sys.modules, which may lead to crash (see details
on stack overflow, link on the ticket)

1 files changed, 2 insertions, 6 deletions

diff --git a/builder.py b/builder.py
index 68bd5c30..eed4ed28 100644
--- a/builder.py
+++ b/builder.py
@@ -121,18 +121,14 @@ class ASTNGBuilder(InspectBuilder):
             raise ASTNGBuildingException(exc)
         except LookupError, exc: # unknown encoding
             raise ASTNGBuildingException(exc)
-        # get module name if necessary, *before modifying sys.path*
+        # get module name if necessary
         if modname is None:
             try:
                 modname = '.'.join(modpath_from_file(path))
             except ImportError:
                 modname = splitext(basename(path))[0]
         # build astng representation
-        try:
-            sys.path.insert(0, dirname(path)) # XXX (syt) iirk
-            node = self.string_build(data, modname, path)
-        finally:
-            sys.path.pop(0)
+        node = self.string_build(data, modname, path)
         node.file_encoding = encoding
         node.file_stream = stream
         return node