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

os x compliance #2

Closed
wants to merge 1 commit into from
Closed

os x compliance #2

wants to merge 1 commit into from

Conversation

rweng
Copy link

@rweng rweng commented Jan 18, 2012

I am currently using littlechef to configure my local machine. rbenv is one of my cookbooks. Although not being used for my local machine, it throws an exception. It seems, the libraries are not automatically included because I do not use the cookbook, but somehow the provider is required. Because the mixin and script_helpers was not required before, the provider throws an exception.

fnichol added a commit that referenced this pull request Jan 23, 2012
@fnichol
Copy link
Contributor

fnichol commented Jan 23, 2012

Right! I had this issue in a couple of other cookbooks due to the way/order that Chef loads in libraries/ code. Are you able to double check if the above patch gets the job done? Thanks!

@rweng
Copy link
Author

rweng commented Jan 23, 2012

This patch does eliminate the error that is thrown when the cookbook is synchronized to a node. It does not, however, make the cookbook run on OS X (at least I did not test that yet, though I am going to do that later).

I used chef-rbenv with this patch on an ubuntu machine, continues to work fine. So it should be safe to merge it in. I might send another pull-request later to provide support for OS X, if necessary.

Fletcher, your cookbooks are among the most stable and well designed cookbooks I've seen, thanks for that.

@fnichol
Copy link
Contributor

fnichol commented Feb 20, 2012

@rweng Thanks for your feedback! I'll be trying this cookbook out on a mac in the next week or so and will try to update with anything missing. The mac should be a first-class supported platform. If you find anything further, just open a new issue and we'll deal with it.

@fnichol fnichol closed this Feb 20, 2012
GUI pushed a commit to NREL-cookbooks/rbenv that referenced this pull request Mar 10, 2013
Create rbenv group before rbenv user
@lock
Copy link

lock bot commented May 31, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants