search

rvm / ubuntu_rvm

Ubuntu package for RVM
https://launchpad.net/~rael-gc/+archive/ubuntu/rvm
Apache License 2.0
694 stars 84 forks source link

Need to load rvm before creating gemsets for sudo users #38

Closed jikamens closed 5 years ago

jikamens commented 5 years ago

The Debian postinst script attempts to call rvm user gemsets within an su command for each user who has sudo rights. This doesn't work when those users haven't already configured rvm because /usr/share/rvm/scripts/rvm doesn't get sources so the rvm command isn't available within the su command. Deal with this by loading the rvm script ourselves before invoking rvm user gemsets.

Fixes rvm/ubuntu_rvm#32

raelgc commented 5 years ago

@jikamens Thank you for the PR! Just to double check: have you tested this solution?

jikamens commented 5 years ago

I tested the modified script directly. I did not test the script packaged within a deb.