Merge pull request #143 from AnouarTouati/parser-windows-path

Fixes: Can't find parser binary on windows when path has spaces
laravel · Jan 3, 2025 · fe8691e · fe8691e
2 parents 69af8e7 + 8b68711
commit fe8691e
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/support/parser.ts b/src/support/parser.ts
@@ -31,7 +31,6 @@ export const setParserBinaryPath = (context: vscode.ExtensionContext) => {
     downloadBinary(context).then((path) => {
         if (path) {
             parserBinaryPath = process.env.PHP_PARSER_BINARY_PATH || path;
-            parserBinaryPath = path.replace(" ", "\\ ");
         }
     });
 };
@@ -170,7 +169,7 @@ const runCommand = (command: string): Promise<string> => {
         }
 
         const extraArgs = os.platform() === "win32" ? "--from-file" : "";
-        const toRun = `${parserBinaryPath} ${command} ${extraArgs}`;
+        const toRun = `"${parserBinaryPath}" ${command} ${extraArgs}`;
 
         // console.log("running command", toRun);