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 apt-get and config file error #16

Closed
wants to merge 2 commits into from
Closed

Fix apt-get and config file error #16

wants to merge 2 commits into from

Conversation

promisedlandt
Copy link

Some minor errors seems to have creeped into this cookbook. Commit should fix them.

If merging these changes in would require me to sign the CLA, please close this pull request and please change the cookbook manually.

@@ -20,6 +20,7 @@
#

include_recipe "erlang"
include_recipe "apt"
Copy link
Contributor

Choose a reason for hiding this comment

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

Could include apt prior to erlang, but not necessary.

@portertech
Copy link
Contributor

+1 cc @jtimberman

@portertech
Copy link
Contributor

@promisedlandt Getting a CLA doesn't take much, just one of those things you should do ;) http://wiki.opscode.com/display/chef/How+to+Contribute

@promisedlandt
Copy link
Author

The problem I have is that I'm signing something with what is not my signature. I believe opscode mentioned something during Chefconf about removing or simplifying that, so I'm biding my time :)

But way offtopic, isn't it? I'd also be happy if you made these changes into a pull request (assuming you've signed the CLA).

@portertech
Copy link
Contributor

@promisedlandt I do have one :) I'll put together a PR in a few moments.

@portertech
Copy link
Contributor

Please refer to #17 (http://tickets.opscode.com/browse/COOK-1551)

@portertech
Copy link
Contributor

@promisedlandt
Copy link
Author

Excellent!

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.

2 participants