From 199904679b30c661770fb70a64c44d094e176de4 Mon Sep 17 00:00:00 2001 From: Piero Toffanin Date: Sat, 23 Aug 2025 13:53:49 -0400 Subject: [PATCH] Revert "Sanitize whitespaces in python invocation" This reverts commit c244f0797b456beb734e40cb7771b3ae33d896d8. --- libs/odmRunner.js | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/libs/odmRunner.js b/libs/odmRunner.js index 5419cfb..5c06375 100644 --- a/libs/odmRunner.js +++ b/libs/odmRunner.js @@ -122,16 +122,13 @@ module.exports = { return; // Skip rest } - const getOdmOptions = (pythonExe, useShell, done) => { + const getOdmOptions = (pythonExe, done) => { // Launch const env = utils.clone(process.env); env.ODM_OPTIONS_TMP_FILE = utils.tmpPath(".json"); env.ODM_PATH = config.odm_path; - const helper = path.join(__dirname, "..", "helpers", "odmOptionsToJson.py"); - const helperSanitized = useShell ? `"${helper}"` : helper; - const odmSanitized = useShell ? `"${config.odm_path}"` : config.odm_path; - let childProcess = spawn(pythonExe, [helperSanitized, - "--project-path", odmSanitized, "bogusname"], { env, stdio: 'inherit', shell: useShell }); + let childProcess = spawn(pythonExe, [path.join(__dirname, "..", "helpers", "odmOptionsToJson.py"), + "--project-path", config.odm_path, "bogusname"], { env, stdio: 'inherit', shell: true }); // Cleanup on done let handleResult = (err, result) => { @@ -163,11 +160,11 @@ module.exports = { } if (os.platform() === "win32"){ - getOdmOptions("helpers\\odm_python.bat", true, done); + getOdmOptions("helpers\\odm_python.bat", done); }else{ // Try Python3 first - getOdmOptions("python3", false, (err, result) => { - if (err) getOdmOptions("python", false, done); + getOdmOptions("python3", (err, result) => { + if (err) getOdmOptions("python", done); else done(null, result); }); }