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

Add Oracle Linux 7.2 #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Oracle Linux 7.2 #29

wants to merge 1 commit into from

Conversation

aogail
Copy link

@aogail aogail commented Feb 22, 2017

Add an Oracle Linux 7.2 box to the list of platforms. My hope is that this is a step toward getting binaries for Oracle Linux on https://rvm.io/binaries/

./run oracle_72_64bit output with these changes: https://gist.github.com/aogail/448692480c34ff3666a51afbdb0a2703

@aogail
Copy link
Author

aogail commented Mar 22, 2017

Is there anything I can do to make this more appealing? ;)

@pkuczynski
Copy link
Member

The thing is that we have md5 sums, but we do not have the files. @mpapis can grant you rights to upload the files or I can run the build again using the image you added in rvm binary.

Copy link
Member

@mpapis mpapis left a comment

Choose a reason for hiding this comment

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

adding to the list +1, ruby requirements should be moved to rvm

[[ -f /etc/os-release ]] &&
GREP_OPTIONS="" \grep 'ID="ol"' /etc/os-release >/dev/null
then
yum install libyaml-devel -y --enablerepo ol7_optional_latest
Copy link
Member

Choose a reason for hiding this comment

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

this should be part of https://github.com/rvm/rvm/blob/master/scripts/functions/requirements/centos, steps here are related to quirks of setting up machines, not configuration for building ruby.

Copy link
Author

Choose a reason for hiding this comment

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

OK, I'll move this. Thanks!

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