-
Notifications
You must be signed in to change notification settings - Fork 449
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
Add freebsd support in packages plugin #814
Conversation
Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog. GitHub Users Who Are Not Authorized To ContributeThe following GitHub users do not appear to have signed a CLA: |
Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future. |
Ping @mcquin |
Ping @thommay as well |
ping @chef/client-freebsd |
I should add the query command works fine on my FreeBSD 10.3 and 9.3 hosts |
So it's been pointed out to me that pkg only exists out of the box on FreeBSD 10. We need to handle the shell out failure better and include some logging. Right now the plugin will fail with a stack trace to debug. |
@tas50 FreeBSD 9.3 does support pkg if setup to use it, otherwise you would use pkgng. You can look at https://github.com/chef/chef/blob/master/lib/chef/resource/freebsd_package.rb to see how the package resource checks for it. |
@Aevin1387 It's not enabled by default though right? |
this is better than nothing and would get support on freebsd 10 + up. should probably make the exitstatus raise if the status is non-zero (and ideally implement shell_out! in the shell_out mixin, because its kind of aggravating not to have the same API in chef and ohai). |
👍 |
2 similar comments
👍 |
👍 |
No description provided.