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

eval $(chef shell init zsh) produces unsightly warnings #3103

Open
trickyearlobe opened this issue Apr 21, 2023 · 3 comments
Open

eval $(chef shell init zsh) produces unsightly warnings #3103

trickyearlobe opened this issue Apr 21, 2023 · 3 comments
Labels
Status: Untriaged An issue that has yet to be triaged.

Comments

@trickyearlobe
Copy link

Description

Nothing serious... just embarrassing on a screen share.
When I launch a terminal window I get the following warnings

/opt/chef-workstation/embedded/lib/ruby/gems/3.1.0/gems/chef-cli-5.6.8/lib/chef-cli/command/shell_init.rb:125: warning: Passing safe_level with the 2nd argument of ERB.new is deprecated. Do not use it, and specify other arguments as keyword arguments.
/opt/chef-workstation/embedded/lib/ruby/gems/3.1.0/gems/chef-cli-5.6.8/lib/chef-cli/command/shell_init.rb:125: warning: Passing trim_mode with the 3rd argument of ERB.new is deprecated. Use keyword argument like ERB.new(str, trim_mode: ...) instead.

In my .zshrc I run eval "$(chef shell-init zsh)"

Chef version: chef-workstation-23.4.1032-1.arm64.dmg
OS Version: Ventura 13.3.1
ZSH Version: zsh 5.9 (arm-apple-darwin22.1.0)
oh-my-zsh: b4f9698
Hardware: MBP with Apple Silicon M2

Originally raised with the CINC team, but seems it's actually in upstream Chef
https://gitlab.com/cinc-project/distribution/workstation/-/issues/29

@trickyearlobe trickyearlobe added the Status: Untriaged An issue that has yet to be triaged. label Apr 21, 2023
@aye2002
Copy link

aye2002 commented Apr 26, 2023

Looks like it's fixed in chef-cli-5.6.11.

@ramereth
Copy link
Contributor

@aye2002 that should be included in the next release of workstation correct?

@nikhil2611
Copy link
Contributor

yes, this change is in chef-cli-5.6.11 and will be included in the upcoming workstation release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Untriaged An issue that has yet to be triaged.
Projects
None yet
Development

No branches or pull requests

4 participants