Fix: use platform-specific separator for AIIDA_PATH in config directory detection (#6935)

Previously, the configuration directory lookup split the AIIDA_PATH environment variable using a hardcoded ':' separator, which caused issues on Windows systems where ';' is used as the path separator. This update introduces platform detection using sys.platform to select the correct separator for splitting paths, ensuring compatibility across operating systems.



 
---------

Co-authored-by: lainme <[email protected]>
Co-authored-by: Ali Khosravi <[email protected]>

Author: 3 people

src/aiida/manage/configuration/settings.py

@@ -12,6 +12,7 @@
 
 import os
 import pathlib
+import sys
 import warnings
 from typing import final
 
@@ -132,9 +133,15 @@ def _get_configuration_directory_from_envvar() -> pathlib.Path | None:
     if environment_variable is None:
         return None
 
+    # Determine seperator to split paths in environment variables like PATH
+    if sys.platform == 'win32':
+        seperator = ';'
+    else:
+        seperator = ':'
+
     # Loop over all the paths in the ``AIIDA_PATH`` variable to see if any of them contain a configuration folder
     dirpath_config = None
-    for base_dir_path in [path for path in environment_variable.split(':') if path]:
+    for base_dir_path in [path for path in environment_variable.split(seperator) if path]:
         dirpath_config = pathlib.Path(base_dir_path).expanduser()
 
         # Only add the base config directory name to the base path if it does not already do so