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

fix reinstallation of inspec if version is already installed #218

Merged
merged 1 commit into from May 3, 2017

Conversation

chris-rock
Copy link
Contributor

@chris-rock chris-rock commented May 3, 2017

fixes #215

This was a tricky one. Once gem is required, it cannot easily been updated. This PR is also avoiding to require inspec for detection.

@chris-rock chris-rock force-pushed the chris-rock/fix-reinstall-of-inspec branch from 3091d55 to 7c06ffd Compare May 3, 2017 10:02
@chris-rock chris-rock changed the title fix reinstallation of inspec if version is already installed WIP: fix reinstallation of inspec if version is already installed May 3, 2017
@chris-rock chris-rock force-pushed the chris-rock/fix-reinstall-of-inspec branch from 7c06ffd to 5c61d35 Compare May 3, 2017 11:18
@chris-rock chris-rock changed the title WIP: fix reinstallation of inspec if version is already installed fix reinstallation of inspec if version is already installed May 3, 2017
@chris-rock chris-rock force-pushed the chris-rock/fix-reinstall-of-inspec branch 6 times, most recently from 3d16d0c to 9721a49 Compare May 3, 2017 19:13
Signed-off-by: Christoph Hartmann <chris@lollyrock.com>
@chris-rock chris-rock force-pushed the chris-rock/fix-reinstall-of-inspec branch from 9721a49 to 7ee2f20 Compare May 3, 2017 19:47
Copy link
Contributor

@arlimus arlimus left a comment

Choose a reason for hiding this comment

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

Great catch!! kudos @chris-rock

@arlimus arlimus merged commit d6bbb2a into master May 3, 2017
@arlimus arlimus deleted the chris-rock/fix-reinstall-of-inspec branch May 3, 2017 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Inspec gem is constantly reinstalled if version is specified
2 participants