fix: update VM configuration handling to use semicolon-separated paths and refactor related structures

xtask/src/ctx.rs

@@ -3,7 +3,7 @@ use ostool::ctx::AppContext;
 pub struct Context {
     pub ctx: AppContext,
     pub build_config_path: Option<std::path::PathBuf>,
-    pub vmconfigs: Vec<String>,
+    pub vmconfigs: String,
 }
 
 impl Context {
@@ -18,7 +18,7 @@ impl Context {
         Context {
             ctx,
             build_config_path: None,
-            vmconfigs: vec![],
+            vmconfigs: String::new(),
         }
     }
 }

xtask/src/main.rs

@@ -44,12 +44,15 @@ enum Commands {
 
 #[derive(Parser)]
 struct QemuArgs {
+    /// Path to the build configuration file
     #[arg(long)]
     build_config: Option<PathBuf>,
+    /// Path to the QEMU configuration file
     #[arg(long)]
     qemu_config: Option<PathBuf>,
+    /// VM configuration files (comma-separated paths)
     #[arg(long)]
-    vmconfigs: Vec<String>,
+    vmconfigs: String,
 }
 
 #[derive(Parser)]
@@ -76,12 +79,15 @@ struct ClippyArgs {
 
 #[derive(Parser)]
 struct UbootArgs {
+    /// Path to the build configuration file
     #[arg(long)]
     build_config: Option<PathBuf>,
+    /// Path to the uboot configuration file
     #[arg(long)]
     uboot_config: Option<PathBuf>,
+    /// VM configuration files (comma-separated paths)
     #[arg(long)]
-    vmconfigs: Vec<String>,
+    vmconfigs: String,
 }
 
 #[tokio::main]

xtask/src/tbuld.rs

@@ -1,4 +1,4 @@
-use std::path::PathBuf;
+use std::{env, path::PathBuf};
 
 use anyhow::Context as _;
 use ostool::build::config::{Cargo, LogLevel};
@@ -45,8 +45,15 @@ impl Context {
 
         self.ctx.build_config_path = Some(config_path);
 
-        let mut vm_configs = config.vm_configs.to_vec();
-        vm_configs.extend(self.vmconfigs.iter().cloned());
+        let vm_configs = if !self.vmconfigs.is_empty() {
+            self.vmconfigs
+                .split(',')
+                .map(|s| s.trim().to_string())
+                .filter(|s| !s.is_empty())
+                .collect()
+        } else {
+            vec![]
+        };
 
         let mut vm_config_paths = vec![];
         for vm_config in &vm_configs {
@@ -78,12 +85,12 @@ impl Context {
         }
 
         if !vm_config_paths.is_empty() {
-            let value = vm_config_paths
-                .iter()
-                .map(|p| format!("{}", p.display()))
-                .collect::<Vec<_>>()
-                .join(";");
-            cargo.env.insert("AXVISOR_VM_CONFIGS".to_string(), value);
+            if let Ok(joined) = env::join_paths(&vm_config_paths) {
+                cargo.env.insert(
+                    "AXVISOR_VM_CONFIGS".to_string(),
+                    joined.to_string_lossy().into_owned()
+                );
+            }
         }
 
         Ok(cargo)