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.

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

Jump to bottom

platform names should be lower case #191

Closed
chris-rock opened this issue Aug 27, 2017 · 0 comments
Closed

platform names should be lower case #191

chris-rock opened this issue Aug 27, 2017 · 0 comments
Labels
Type: Bug Feature not working as expected

Comments

@chris-rock
Copy link
Contributor

Running detect on some operating systems returns upper case values:

inspec detect -t ssh://[email protected]:2200 -i /Users/chartmann/Development/tests/freebsd-11/.vagrant/machines/default/virtualbox/private_key 

== Operating System Details

Name:      FreeBSD
Family:    freebsd
Release:   11.1-RELEASE-p1
Arch:

This leads to wrong behavior where we do checks like

%w{freebsd}.include?(os[:name])
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Bug Feature not working as expected
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

return lowercase for platform name on bsd, esx and windows inspec/train
3 participants
@clintoncwolfe @jquick @chris-rock