Fix multiple path bug in windows

Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>

Author: AyanSinhaMahapatra

configure.bat

@@ -163,7 +163,7 @@ if %ERRORLEVEL% neq 0 (
     %FLOT_REQUIREMENTS%
 
 "%CFG_BIN_DIR%\flot" --pyproject pyproject-commoncode.toml
-"%CFG_BIN_DIR%\pip" install ./dist/commoncode*.whl
+"%CFG_BIN_DIR%\pip" install "%CFG_BIN_DIR%\dist\commoncode-32.5.2-py3-none-any.whl"
 
 "%CFG_BIN_DIR%\pip" install ^
     --upgrade ^

src/commoncode/resource.py

@@ -49,6 +49,7 @@
 from commoncode.fileutils import file_name
 from commoncode.fileutils import parent_directory
 from commoncode.fileutils import splitext_name
+from commoncode.system import to_os_native_path
 
 """
 This module provides Codebase and Resource objects as an abstraction for files
@@ -64,7 +65,7 @@
 
 # Tracing flags
 TRACE = False
-TRACE_DEEP = False
+TRACE_DEEP = True
 
 
 def logger_debug(*args):
@@ -586,7 +587,7 @@ def _create_resources_from_root(self, root, includes, ignores):
         # track resources parents by location during construction.
         # NOTE: this cannot exhaust memory on a large codebase, because we do
         # not keep parents already walked and we walk topdown.
-        parents_by_loc = {root.location: root}
+        parents_by_loc = {to_os_native_path(root.location): root}
 
         def err(_error):
             """os.walk error handler"""
@@ -613,7 +614,10 @@ def err(_error):
                 includes=includes,
             ):
                 if not created.is_file:
-                    parents_by_loc[created.location] = created
+                    parents_by_loc[to_os_native_path(created.location)] = created
+
+        if TRACE_DEEP:
+            logger_debug(f"parents_by_loc: {parents_by_loc}")
 
         # we start walking through all the input locations
         for included_location in includes:
@@ -624,7 +628,12 @@ def err(_error):
                 max_depth=self.max_depth,
                 error_handler=err,
             ):
-                parent = parents_by_loc.pop(top)
+                if TRACE_DEEP:
+                    logger_debug(f"parents_by_loc: {parents_by_loc}")
+                try:
+                    parent = parents_by_loc.pop(top)
+                except KeyError:
+                    raise Exception(parents_by_loc, includes, root.location, )
                 for created in self._create_resources(
                     parent=parent,
                     top=top,
@@ -634,7 +643,7 @@ def err(_error):
                 ):
                     # on the plain, bare FS, files cannot be parents
                     if not created.is_file:
-                        parents_by_loc[created.location] = created
+                        parents_by_loc[to_os_native_path(created.location)] = created
 
     def _create_resources(self, parent, top, dirs, files, skip_ignored=skip_ignored):
         """

src/commoncode/system.py

@@ -13,6 +13,15 @@
 from commoncode.distro import parse_os_release
 
 
+def to_os_native_path(path):
+    """
+    Normalize a path to use the native OS path separator.
+    """
+    OS_PATH_SEP = "\\" if on_windows else "/"
+
+    return path.replace("/", OS_PATH_SEP).replace("\\", OS_PATH_SEP).rstrip(OS_PATH_SEP)
+
+
 def os_arch():
     """
     Return a tuple for the current the OS and architecture.

src/commoncode/testcase.py

@@ -29,7 +29,7 @@
 from commoncode.archive import extract_zip_raw
 from commoncode.archive import tar_can_extract  # NOQA
 from commoncode.system import on_posix
-from commoncode.system import on_windows
+from commoncode.system import to_os_native_path
 
 # a base test dir specific to a given test run
 # to ensure that multiple tests run can be launched in parallel
@@ -39,15 +39,6 @@
 timing_threshold = sys.maxsize
 
 
-def to_os_native_path(path):
-    """
-    Normalize a path to use the native OS path separator.
-    """
-    OS_PATH_SEP = "\\" if on_windows else "/"
-
-    return path.replace("/", OS_PATH_SEP).replace("\\", OS_PATH_SEP).rstrip(OS_PATH_SEP)
-
-
 def get_test_loc(
     test_path,
     test_data_dir,