/etc/profile.d/chruby breaks mdm upstart script

[bemurphy]

Still trying to figure out exactly what is up, but I discovered that mdm on Linux Mint 15 can't start when /etc/profile.d/chruby is in place. Moved it out, and all was fine.

Not sure exactly if this is chruby or mdm problem, still hunting.

[postmodern]

Did you add [ -n "$BASH_VERSION" ] || [ -n "$ZSH_VERSION" ] || return to the top?

[bemurphy]

the /etc/profile.d/chruby file was generated from scripts/setup.sh, so it had that at the top, yes.

[vovimayhem]

Damn after like 10 reinstalls of Linux Mint 16 during the past 4 weeks, I finally pinpointed this file was the problem...

I had the following contents:

[ -n "$BASH_VERSION" ] || [ -n "$ZSH_VERSION" ] || return

source /usr/local/share/chruby/chruby.sh
source /usr/local/share/chruby/auto.sh

I went commenting each line upward... It failed to start the MDM even when i left just the first line uncommented...

I'll see if I can make my way through this.

[vovimayhem]

I ended changing my file to this:

if [ -n "$BASH_VERSION" -o -n "$ZSH_VERSION" ]; then
    source /usr/local/share/chruby/chruby.sh
    source /usr/local/share/chruby/auto.sh
fi

Also I found this other issue that talked about this: #56

Any comments???

[postmodern]

I updated the README instructions to use if [ -n "$BASH_VERSION" ] || [ -n "$ZSH_VERSION" ]; then. Does that fix the issue?

[vovimayhem]

Nice!!!

It solves the issue on my lap (Linux MInt 16 64bit). I don't see how it would cause problems on other distributions, so it seems pretty fair to me!

Thanks!

[postmodern]

I suspect they are loading the files oddly (maybe using eval?) and the explicit return is breaking the the control flow. I think we can close this now.