Skip to content
This repository has been archived by the owner on Jul 14, 2021. It is now read-only.

chef shell-init should support powershell and cmd #186

Closed
bwmeier opened this issue Oct 8, 2014 · 5 comments
Closed

chef shell-init should support powershell and cmd #186

bwmeier opened this issue Oct 8, 2014 · 5 comments
Labels
Platform: Windows

Comments

@bwmeier
Copy link

bwmeier commented Oct 8, 2014

This requires adopting a fix to issue #180 to ensure that the PATH is correct on windows. I have a patch to handle the windows shells that relies on that fix.
bwmeier added a commit to bwmeier/chef-dk that referenced this issue Oct 8, 2014
@bwmeier
Patch to chef-dk to resolve issue chef-boneyard#186
652af1d 
- Add powershell and cmd to chef shell-init
- Handle stray quotes in powershell
- Powershell documentation in help message
@sersut sersut added the Windows label Oct 8, 2014
@danielsdeleo
Copy link
Contributor

PR #187 will fix this

@bwmeier bwmeier mentioned this issue Nov 3, 2014
Closed
@sersut sersut added the Priority label Nov 5, 2014
btm pushed a commit that referenced this issue Nov 13, 2014
@bwmeier @btm
Patch to chef-dk to resolve issue #186
666c2ed 
- Add powershell and cmd to chef shell-init
- Handle stray quotes in powershell
- Powershell documentation in help message
@danielsdeleo
Copy link
Contributor

Powershell is supported now.

@joelhandwell
Copy link

@danielsdeleo is it decided to not to support cmd? Today I got following:

joel@DESKTOP-90SRLRU D:\code\oss\haapp
> chef shell-init cmd
Shell `cmd' is not currently supported
Supported shells are: bash fish zsh sh powershell posh

@danielsdeleo
Copy link
Contributor

We'd probably accept a pull request for cmd support, though I'd try to steer folks to powershell since the experience is so much better. But I understand it may not always make sense to install it on ci nodes or whatever

@coderanger
Copy link
Contributor

Does cmd.exe even support the kind of eval behavior needed to make shell-init work?

@chef-boneyard chef-boneyard locked and limited conversation to collaborators Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Platform: Windows
Milestone
No milestone
Development

No branches or pull requests
6 participants
@danielsdeleo @coderanger @tas50 @sersut @bwmeier @joelhandwell