Change ";" to os.pathsep

features/environment.py

@@ -51,10 +51,8 @@ def _setup_context_with_venv(context, venv_dir):
     # this is because exe resolution in subprocess doesn't respect a passed env
     if os.name == "posix":
         bin_dir = context.venv_dir / "bin"
-        path_sep = ":"
     else:
         bin_dir = context.venv_dir / "Scripts"
-        path_sep = ";"
     context.bin_dir = bin_dir
     context.pip = str(bin_dir / "pip")
     context.python = str(bin_dir / "python")
@@ -63,11 +61,11 @@ def _setup_context_with_venv(context, venv_dir):
 
     # clone the environment, remove any condas and venvs and insert our venv
     context.env = os.environ.copy()
-    path = context.env["PATH"].split(path_sep)
+    path = context.env["PATH"].split(os.pathsep)
     path = [p for p in path if not (Path(p).parent / "pyvenv.cfg").is_file()]
     path = [p for p in path if not (Path(p).parent / "conda-meta").is_dir()]
     path = [str(bin_dir)] + path
-    context.env["PATH"] = path_sep.join(path)
+    context.env["PATH"] = os.pathsep.join(path)
 
     # Create an empty pip.conf file and point pip to it
     pip_conf_path = context.venv_dir / "pip.conf"

kedro/framework/startup.py

@@ -142,7 +142,7 @@ def _add_src_to_path(source_dir: Path, project_path: Path) -> None:
 
     python_path = os.getenv("PYTHONPATH") or ""
     if str(source_dir) not in python_path:
-        sep = ";" if python_path else ""
+        sep = os.pathsep if python_path else ""
         os.environ["PYTHONPATH"] = f"{str(source_dir)}{sep}{python_path}"