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

added 'append_system_gemsets' feature for multiuser mode #3473

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

Conversation

kvechera
Copy link

Hello.

I've implemented mixing user's and system gemsets for multi-user mode. It allows to use a large set of system-wide gems with a compact user-specific gem sets. Please take a look.

It appends system gems directories (from ruby's primary gem home and '@global') to the GEM_PATH environment variable and the directories of gem's binaries to the PATH, allowing to use user installed gems and system gems together.

The feature adds '--append-system-gemsets' argument to the 'rvm user' command and saves 'rvm_append_system_gemsets' flag in the user's rvmrc file. Than If the flag is on, it appends system gems pathes to GEM_PATH and PATH.


if (( rvm_append_system_gemsets ))
then
GEM_PATH="$GEM_PATH:$rvm_ruby_gem_system_path"
Copy link
Member

Choose a reason for hiding this comment

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

no need for this, rvm_ruby_gem_path should be updated, see bellow

@mpapis
Copy link
Member

mpapis commented Feb 25, 2016

left some notes how to make it better, also would you be able to update https://github.com/rvm/rvm/blob/master/help/user.md?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants