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

[plans] Add a ruby plan #138

Merged
merged 1 commit into from
Jan 12, 2016
Merged

[plans] Add a ruby plan #138

merged 1 commit into from
Jan 12, 2016

Conversation

jtimberman
Copy link
Contributor

This also adds dependencies for ruby:

  • libffi
  • libiconv
  • libtool
  • libyaml

We needed to add the patch ruby-2_1_3-no-mkmf to prevent errors in make install. We also needed to force the openssl and libyaml directories by using the _resolve_dependency function from bldr-build.

This also adds dependencies for ruby:

* libffi
* libiconv
* libtool
* libyaml

We needed to add the patch ruby-2_1_3-no-mkmf to prevent errors in
make install. We also needed to force the openssl and libyaml
directories by using the _resolve_dependency function from bldr-build.
@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

1 similar comment
@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

@reset
Copy link
Collaborator

reset commented Jan 12, 2016

@jtimberman looks very clean!

@delivery approve

chef-delivery added a commit that referenced this pull request Jan 12, 2016
Merged change 467cfa25-2b4a-40d3-a18d-304d0ba2cf5a

From review branch jtimberman/ruby-plan into master

Signed-off-by: reset <[email protected]>
@chef-delivery chef-delivery merged commit ec6a578 into master Jan 12, 2016
@chef-delivery
Copy link
Contributor

Change: 467cfa25-2b4a-40d3-a18d-304d0ba2cf5a approved by: @reset

@chef-delivery chef-delivery deleted the jtimberman/ruby-plan branch January 12, 2016 22:30
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.

3 participants