Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update to electron 30 #215406

Merged
merged 14 commits into from
Jul 9, 2024
Prev Previous commit
Next Next commit
fix: use shell when spawning .bat or .cmd files
  • Loading branch information
deepak1556 committed Jul 9, 2024
commit fdf4449559d2d063430c901e9a39c20d85b8fd8e
8 changes: 4 additions & 4 deletions extensions/vscode-test-resolver/src/extension.ts
Original file line number Diff line number Diff line change
Expand Up @@ -164,8 +164,8 @@ export function activate(context: vscode.ExtensionContext) {
const serverCommandPath = path.join(vscodePath, 'scripts', serverCommand);

outputChannel.appendLine(`Launching server: "${serverCommandPath}" ${commandArgs.join(' ')}`);

extHostProcess = cp.spawn(serverCommandPath, commandArgs, { env, cwd: vscodePath });
const shell = (process.platform === 'win32');
extHostProcess = cp.spawn(serverCommandPath, commandArgs, { env, cwd: vscodePath, shell });
} else {
const extensionToInstall = process.env['TESTRESOLVER_INSTALL_BUILTIN_EXTENSION'];
if (extensionToInstall) {
Expand All @@ -182,8 +182,8 @@ export function activate(context: vscode.ExtensionContext) {

outputChannel.appendLine(`Using server build at ${serverLocation}`);
outputChannel.appendLine(`Server arguments ${commandArgs.join(' ')}`);

extHostProcess = cp.spawn(path.join(serverLocation, 'bin', serverCommand), commandArgs, { env, cwd: serverLocation });
const shell = (process.platform === 'win32');
extHostProcess = cp.spawn(path.join(serverLocation, 'bin', serverCommand), commandArgs, { env, cwd: serverLocation, shell });
}
extHostProcess.stdout!.on('data', (data: Buffer) => processOutput(data.toString()));
extHostProcess.stderr!.on('data', (data: Buffer) => processOutput(data.toString()));
Expand Down