Add prefix after all configure arguments

Author: Pencilcaseman

Cargo.lock

@@ -1,7 +1,7 @@
 [package]
 name = "sccmod"
 authors = ["Toby Davis \"Pencilcaseman\""]
-version = "0.3.2"
+version = "0.3.3"
 edition = "2021"
 readme = "README.md"
 license = "MIT OR Apache-2.0"

Cargo.toml

@@ -37,26 +37,18 @@ impl Make {
             shell.add_command(&format!("module load {dep}"));
         }
 
-        let mut configure_cmd = format!("{source_path:?}/configure --prefix={install_path:?}");
+        let mut configure_cmd = format!("{source_path:?}/configure");
 
         if let Some(flags) = &self.configure_flags {
             for flag in flags {
                 configure_cmd.push_str(&format!(" {flag}"));
             }
         }
 
-        shell.add_command(&configure_cmd);
-
-        // configure.stdout(std::process::Stdio::piped());
-        // configure.stderr(std::process::Stdio::piped());
+        // Add this last so it overrides anything passed in `configure_flags`
+        configure_cmd.push_str(&format!("--prefix={install_path:?}"));
 
-        // let spawn = configure.spawn().map_err(|e| e.to_string())?;
-        // let (result, stdout, stderr) = child_logger(spawn);
-
-        // if result.is_err() {
-        //     return Err("Failed to run ./configure".to_string());
-        // }
-        // let result = result.unwrap();
+        shell.add_command(&configure_cmd);
 
         let (result, stdout, stderr) = shell.exec();
         let result = result.map_err(|_| "Failed to run ./configure")?;