Fix issue #53: use package target instead of default-target for path

The bug was in config.py line 103, which used the 'target' variable
(always equal to default-target) instead of package["target"]
(which may contain a custom per-package target setting).

This fix ensures that:
- Packages with custom target settings are checked out to their
  specified directory
- Packages without custom target continue to use default-target
- Variable interpolation like ${settings:docs-directory} works correctly

The previously failing test now passes, demonstrating the fix works.

Fixes #53

Author: jensens

CHANGES.md

@@ -2,6 +2,9 @@
 
 ## 4.1.2 (unreleased)
 
+- Fix #53: Per-package target setting now correctly overrides default-target when constructing checkout paths.
+  [jensens]
+
 - Fix #55: UnicodeEncodeError on Windows when logging emoji. The emoji is now conditionally displayed only when the console encoding supports it (UTF-8), avoiding errors on Windows cp1252 encoding.
   [jensens]
 

src/mxdev/config.py

@@ -100,7 +100,8 @@ def is_ns_member(name) -> bool:
             package.setdefault("install-mode", mode)
             package.setdefault("vcs", "git")
             # XXX: path should not be necessary in WorkingCopies
-            package.setdefault("path", os.path.join(target, name))
+            # Use package["target"] not 'target' variable to respect per-package target setting (#53)
+            package.setdefault("path", os.path.join(package["target"], name))
             if not package.get("url"):
                 raise ValueError(f"Section {name} has no URL set!")
             if package.get("install-mode") not in ["direct", "skip"]: