Better RVM & Passenger Integration

[RLovelett]

Allows Passenger to read and use an RVM .rvmrc file if it exists in the directory.

[robbkidd]

+1 We've added this to our install as well. Content comes straight from RVM docs for Passenger.

[ariejan]

This is great if you use passenger and rvm. The installation guide sets you up with unicorn and nginx.

It'd be great to add this to the wiki for those who need it.

Also, Passenger 3 has awesome rvm/multi-rubies support: http://blog.phusion.nl/2010/09/21/phusion-passenger-running-multiple-ruby-versions/

[robbkidd]

Thanks for your response, @ariejan.

I do not believe this change is specific to Passenger; I believe that it is needed for RVM.

We are using nginx and Unicorn. We followed the installation guide except that we chose to manage our Ruby install with RVM instead of downloading and installing directly from source. We had to make the change described in this pull request to our install even though we are not using Apache or Passenger.

This would be a nice thing to include in the code base for users who opt to use RVM. It should be safe to include for those users who follow the install instructions directly. But if GitLabHQ does not wish to support RVM out of the box, the wiki--as you say--would be a great place to document this.

[brodock]

+1 to get this merged...

also:

I want to point to one more thing that can be changed that will benefit thous using RVM without any side effect:

Instead of suggesting to run commands as:

sudo -u gitlab bundle exec ...

It's better to run it as:

sudo su - gitlab bash -c 'cd gitlab;bundle exec ...'

The reason for this is that RVM depends on scripts that are loaded during "login", and will not be available otherway.

And finally at the init script, change:

sudo -u gitlab sh -c "$CD_TO_APP_DIR > /dev/null 2>&1 && $START_DAEMON_PROCESS && $START_RESQUE_PROCESS"

to:

sudo su - gitlab bash -c "$CD_TO_APP_DIR > /dev/null 2>&1 && $START_DAEMON_PROCESS && $START_RESQUE_PROCESS"

[riyad]

+1