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 support for passenger >= 4 #85

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added support for passenger >= 4 #85

wants to merge 2 commits into from

Conversation

pspoerri
Copy link
Contributor

The install script for passenger >= expects the home variable to be set. Choosing /tmp
since we use it from puppet.

Error:

/usr/local/rvm/gems/ruby-1.9.3-p429/gems/passenger-4.0.5/lib/phusion_passenger/platform_info/ruby.rb:202:in `expand_path': non-absolute home (ArgumentError)

Also added a symlink since passenger changed the bin directory from

 ${gempath}/passenger-${version}/ext/

to

${gempath}/passenger-${version}/libout/

pspoerri added 2 commits June 11, 2013 11:50
The install script expects the home variable to be set. Choosing /tmp
since we use it from puppet.

Error:
/usr/local/rvm/gems/ruby-1.9.3-p429/gems/passenger-4.0.5/lib/phusion_passenger/platform_info/ruby.rb:202:in `expand_path': non-absolute home (ArgumentError)
Sets a symlink with exec when the destination folder
    ${gempath}/passenger-${version}/ext/
does not exist. Also adapted compilation of library.
exec {
'create-symlink-for-passenger-4.0-modules':
command => "ln -s ${gempath}/passenger-${version}/libout/apache2/mod_passenger.so ${gempath}/passenger-${version}/ext/apache2/mod_passenger.so",
unless => "test -e ${gempath}/passenger-${version}/ext",
Copy link

Choose a reason for hiding this comment

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

On my system, the ext directory exists, causing this symlink to not be created. The mod_passenger.so does exist in the buildout directory. Can we just change the generated Apache config to point to that if the version is >= 4.0.0?

Choose a reason for hiding this comment

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

I did a pull request solving this in the template file for the Apache conf.d file (based on what was already there for the previous "ext" to "libout" changes). That may be a better method as it just uses the correct path in the LoadModule line. https://github.com/blt04/puppet-rvm/pull/89/files

@germs12
Copy link

germs12 commented Jan 8, 2014

 *** EXCEPTION: non-absolute home (ArgumentError)
Notice: /Stage[main]/Rvm::Passenger::Apache::Ubuntu::Post/Exec[passenger-install-apache2-module]/returns:     /usr/local/rvm/gems/ruby-2.0.0-p353/gems/passenger-4.0.33/lib/phusion_passenger/platform_info/ruby.rb:221:in `expand_path'

I am also experiencing this issue when I attempt to install passenger. I'm on Ubuntu 12.04, Ruby 2.0.0p353, Passenger 4.0.33. Any ideas how I might get around this?

@germs12
Copy link

germs12 commented Jan 8, 2014

I cloned the repo from @inetdavid and it worked.

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

Successfully merging this pull request may close these issues.

4 participants