Skip to content

Commit 1a6dc33

Browse files
committed
vamos_runner: Fix AmiPath validation failure on Windows
The cwd and command path configs embedded the host temp directory path directly into Amiga-style "volume:path" notation, producing strings like "root:C:\Users\...\Temp\amifuse_XXXXX". The amitools AmiPath validator explicitly rejects paths containing more than one colon, so the drive letter colon in "C:\" caused path manager setup to fail on Windows. Use "root:" (the volume root) instead. The volume definition already maps root: to the host temp directory, so the resolved path is identical.
1 parent e466d16 commit 1a6dc33

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

amifuse/vamos_runner.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,13 @@ def setup(self, cpu: Optional[str] = None):
9191
path_cfg["path"]["auto_volumes"] = []
9292
path_cfg["path"]["auto_assigns"] = []
9393
path_cfg["path"]["vols_base_dir"] = temp_path
94-
path_cfg["path"]["cwd"] = f"root:{temp_path}"
95-
path_cfg["path"]["command"] = [f"root:{temp_path}"]
94+
# Use "root:" (volume root) for cwd/command rather than embedding
95+
# the host path. On Windows the host temp path contains a drive
96+
# letter colon (e.g. C:\…) which is rejected by AmiPath validation
97+
# (only one colon allowed). The volume definition below maps root:
98+
# to the actual temp directory, so "root:" resolves correctly.
99+
path_cfg["path"]["cwd"] = "root:"
100+
path_cfg["path"]["command"] = ["root:"]
96101
# Provide a minimal root: volume pointing to temp dir so cwd resolves.
97102
path_cfg["volumes"] = [f"root:{temp_path}"]
98103
path_cfg["assigns"] = {}

0 commit comments

Comments
 (0)