Fix sudo-less GEM_HOME detection #188
Conversation
When installing sudo-less the GEM_HOME var was no longer accurate, at least not for me on macOS Catalina. To avoid such errors automate the GEM_HOME discovery.
| @@ -51,8 +51,8 @@ tell RubyGems to install into your user directory by passing either the | |||
| The latter is in our opinion the best solution. To do this open up terminal and create or edit your `.bash_profile` with your preferred editor. Then enter these lines into the file: | |||
|
|
|||
| ```shell | |||
| export GEM_HOME=$HOME/.gem | |||
| export PATH=$GEM_HOME/bin:$PATH | |||
| export GEM_HOME="$(ruby -r rubygems -e 'puts Gem.user_dir')" | |||
There was a problem hiding this comment.
I don't think we want to suggest this because it would run ruby every time the profile is loaded.
$HOME/.gem looks correct to me except it doesn't have the ruby/[VERSION] suffix, but I don't personally know if that's required when setting GEM_HOME
There was a problem hiding this comment.
If you use the builtin ruby version on macOS, that is the only thing that works automagically. The only solution would be to type the version in by hand. I personally see no significant delay when my profile is loaded, and i am also running python and some other scripts in my profile amongst this one ruby call. I only load my profile after a reboot or if i quit the terminal or open a new window, things i do rarely. I think this approach will also benefit people who use rbenv to have multiple versions of ruby.
I wanted to give some reasoning and context, ultimately it's your decision, please close the PR if there is no further action needed.
Thank you.
There was a problem hiding this comment.
I think we probably want to switch, I just poked around on a clean machine and realistically this is the only way to pull it out.
An alternative, docs wise, is to offer the exact folders for OS versions, e.g. export GEM_HOME=.gem/ruby/2.6.0 for macOS 10.15+
When installing sudo-less the GEM_HOME var was no longer accurate, at least not for me on macOS Catalina. To avoid such errors automate the GEM_HOME discovery.