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

Fixed windows case to prevent bad java_home variable setting #106

Merged
merged 1 commit into from Dec 18, 2013

Conversation

Projects
None yet
2 participants
@TheSultan

TheSultan commented Dec 18, 2013

I encountered a bug in this version which caused a bad java_home node attribute to be set because of the case statement below (it would fall through to the else). That in turn caused a problem with the generated command which caused a popup, hanging chef-client runs. Below is a quick fix.

carmstrong added a commit that referenced this pull request Dec 18, 2013

Merge pull request #106 from TheSultan/master
Fixed windows case to prevent bad java_home variable setting

@carmstrong carmstrong merged commit 069c4ad into sous-chefs:master Dec 18, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment