-
-
Notifications
You must be signed in to change notification settings - Fork 137
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 RHEL 7 & 8 based distros to work around /tmp execution issues #230
Conversation
Shout out to @coofercat for the suggested fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
I could use this as well - thanks! |
Thought I'd follow-up on this to see if it might be able to be merged (I came across my #TODO in my galaxy file preventing me from being able to enable the galaxy collection for rvm until this is merged in). |
@stevenhaddox Fantastic, thank you for the quick change. I'm terribly sorry to ping-pong with another request, but could you add a brief entry in CHANGELOG.md? Just a bullet would about fixing installation error in CentOS 7&8 would do. |
@sfgeorge no need to apologize. Thanks for your patience. Hopefully that last push is what you're looking for 🤞🏻 |
I shall bother you no more, this hungry commit-monster is finally satiated! Thank you for the great fix, @stevenhaddox ! |
Oops, I don't have ability to move this forward. @danochoa Can you merge if you deem it ready? |
It looks like the old CI server is no longer working. Maybe @pkuczynski can help? I think I don't have access to fix this but I can poke around a bit later. https://travis-ci.org/github/rvm/rvm1-ansible/requests |
Thanks! |
Implements the suggested fix from #155 in order to enable rvm to install properly on CentOS 8 (and hopefully 7 and a few other distro variations as well).
Fixes #155