postmodern / chruby

Changes the current Ruby
MIT License
2.86k stars 190 forks source link

Disable chruby_auto if RUBY_AUTO_DISABLE is set #338

Closed sheldonh closed 9 years ago

sheldonh commented 9 years ago

Fixes postmodern/chruby#337

postmodern commented 9 years ago

Why can't you simply not load auto.sh, or is another program loading both by default? It's a separate file specifically to allow users to opt-out.

sheldonh commented 9 years ago

I can edit the /etc/profile.d/chruby.sh that the setup helper writes. It's just root-owned, so I figured it would be nice for individual users to have a way to opt out.

postmodern commented 9 years ago

@sheldonh which setup helper? Perhaps we could make it more configurable.

sheldonh commented 9 years ago

Oh, sorry. I'm referring to the setup.sh script referred to from the README.

postmodern commented 9 years ago

@sheldonh ah hmm. Did someone else run the script and you don't have sudo privileges?

sheldonh commented 9 years ago

Nope. I ran the script and I have root privilege. As the only user on my workstation, it's a non-issue. I can just edit the file (or not run it in the first place, sourcing just chruby.sh in my personal .bashrc).

My thinking went like this:

If you don't like the idea, you can bin it. I won't be offended. Chruby is a great little tool. Thanks.

postmodern commented 9 years ago

I'm thinking of adding options to setup.sh to enable/disable things. --no-auto or maybe --no-global.