Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

get rid of rubygem-bundler as run-time requirement #277

Merged
merged 1 commit into from
Aug 18, 2015
Merged

get rid of rubygem-bundler as run-time requirement #277

merged 1 commit into from
Aug 18, 2015

Conversation

jordimassaguerpla
Copy link
Member

If we install bundler gem under vendor/bundle/ruby/2.1.0 directory,
and since we had that directory as GEM_PATH in the vhost configuration,
we don't need rubygem-bundler rpm as a runtime requirement.

Bundler does not install itself if you add it at the Gemfile, thus we
need to install it with "gem install --install-dir ..."

If we install bundler gem under vendor/bundle/ruby/2.1.0 directory,
and since we had that directory as GEM_PATH in the vhost configuration,
we don't need rubygem-bundler rpm as a runtime requirement.

Bundler does not install itself if you add it at the Gemfile, thus we
need to install it with "gem install --install-dir ..."
@jordimassaguerpla
Copy link
Member Author

It turned out to be much simpler than what I expected ...

@@ -81,6 +79,10 @@ APPLICATION_CSS=$(find . -name application-*.css 2>/dev/null)
cp $APPLICATION_CSS public/landing.css

bundle install --retry=3 --local --deployment
Copy link
Member

Choose a reason for hiding this comment

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

Do we still need this line?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes we do.

@flavio
Copy link
Member

flavio commented Aug 18, 2015

LGTM, I just made a small comment.

jordimassaguerpla added a commit that referenced this pull request Aug 18, 2015
get rid of rubygem-bundler as run-time requirement
@jordimassaguerpla jordimassaguerpla merged commit 07063df into SUSE:master Aug 18, 2015
@jordimassaguerpla jordimassaguerpla deleted the remove_bundle_requirement branch August 18, 2015 12:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants