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

remove runtime dependency of vmstat #25

Conversation

majormoses
Copy link
Member

This is to address: #22

I wish there was a better way to do this but I cant think of any. Open to any suggestions, this is more of a proposal than being ready to go.

I have not tested and will be kind of harder for me to test because I would have to custom build something from scratch because all our automation has gcc installed via the build-essentials package.

Also the reason it is a full release is because its a breaking change with no backwards compatibility, rather than a minor, or patch.

Updated doc and check to warn user about change in behavior.
@majormoses majormoses force-pushed the feature/remove-vmstat-runtime-dep branch from 7e2ec15 to 7bd9fed Compare March 6, 2016 20:32
analytically added a commit that referenced this pull request Mar 6, 2016
@analytically analytically merged commit 9bf6380 into sensu-plugins:master Mar 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants