Skip to content

child-process: support any shells on Windows #21943

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.

Sign up for GitHub

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

Closed
tkamenoko wants to merge 16 commits into from
Closed

child-process: support any shells on Windows #21943

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
b42be28
child-process: fix switches for alternative shells on Windows
tkamenoko Jul 21, 2018
e9863e6
child-process: fix lint errors
tkamenoko Jul 21, 2018
6f3bccc
test: add exec-any-shells test
tkamenoko Jul 21, 2018
6b9b090
test: fix exec-any-shell test for powershell
tkamenoko Jul 23, 2018
1dcea34
child-process: enable to accept full path that contains spaces
tkamenoko Jul 23, 2018
c7163f8
child-process: undo unrelated style changes
tkamenoko Jul 24, 2018
a314340
test: remove hard coding path
tkamenoko Jul 26, 2018
ce7999a
test: fix spawnsync-shell to check correct options on Windows
tkamenoko Jul 26, 2018
097c905
child-process: set windowsVerbatimArguments true for any shell
tkamenoko Aug 2, 2018
a4122b7
child-process: use case-insensitive comparison to check shell name
tkamenoko Aug 2, 2018
27ba48f
test: add testcases to ensure uppercase shell name and complex commands
tkamenoko Aug 2, 2018
f93c796
test: fix to accept multiple lines output of `where bash`
tkamenoko Aug 2, 2018
ade01d9
test: revert windowsVerbatim flag, fix `isCmd` check in spawnsync-shell
tkamenoko Aug 2, 2018
5a2c130
doc: update shell requirements in child_process
tkamenoko Aug 7, 2018
e2192a3
fixup: windowsVerbatimArguments
joaocgreis Aug 14, 2018
7a10600
fixup: don't exhaust exec buffer
joaocgreis Sep 1, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fixup: don't exhaust exec buffer
  • Loading branch information
@joaocgreis
joaocgreis committed Sep 1, 2018
commit 7a10600f990d18eef5f65a89dc7c6d908db352cc
16 changes: 10 additions & 6 deletions test/parallel/test-child-process-exec-any-shells-windows.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,16 @@ test('CMD');
test('powershell');
Copy link
Copy Markdown
Member

@joaocgreis joaocgreis Jul 31, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also include a more complex PowerShell test (with quotes and pipes)? Something like

cp.exec(`Get-ChildItem "${__dirname}" | Select-Object -Property Name`,
        { shell: 'PowerShell' }, (error, stdout, stderror) => {
  assert.ok(!error && !stderror);
  assert.ok(stdout.includes('test-child-process-exec-any-shells-windows.js'));
});

testCopy('powershell.exe',
`${system32}\\WindowsPowerShell\\v1.0\\powershell.exe`);
cp.exec(`Get-ChildItem "${__dirname}" | Select-Object -Property Name`,
{ shell: 'PowerShell' }, common.mustCall((error, stdout, stderror) => {
assert.ok(!error && !stderror);
assert.ok(stdout.includes(
'test-child-process-exec-any-shells-windows.js'));
}));
fs.writeFile(`${tmpPath}\\test file`, 'Test', common.mustCall((err) => {
assert.ifError(err);
cp.exec(`Get-ChildItem "${tmpPath}" | Select-Object -Property Name`,
{ shell: 'PowerShell' },
common.mustCall((error, stdout, stderror) => {
assert.ok(!error && !stderror);
assert.ok(stdout.includes(
'test file'));
}));
}));

// Test Bash (from WSL and Git), if available
cp.exec('where bash', common.mustCall((error, stdout) => {
Expand Down