use erb trim mode '-' to skip new lines + make sure GEM_PATH is set by default

forcing `ENV['GEM_PATH'] = ENV['GEM_HOME']` (closes GH-344)

Author: kares

lib/warbler/jar.rb

@@ -272,7 +272,7 @@ def add_with_pathmaps(config, f, map_type)
 
     def expand_erb(file, config)
       require 'erb'
-      erb = ERB.new(File.open(file) {|f| f.read })
+      erb = ERB.new(File.read(file), nil, '-')
       StringIO.new(erb.result(erb_binding(config)))
     end
 

lib/warbler/templates/bundler.erb

@@ -1,2 +1,4 @@
 ENV['BUNDLE_WITHOUT'] = '<%= config.bundle_without.join(':') %>'
-<% if config.bundler[:frozen] %>ENV['BUNDLE_FROZEN'] = '1'<% end %>
+<% if config.bundler[:frozen] -%>
+ENV['BUNDLE_FROZEN'] = '1'
+<% end -%>

lib/warbler/templates/jar.erb

@@ -1,9 +1,11 @@
-<% assignment_operator = config.override_gem_home ? "=" : "||=" %>
-<% if config.relative_gem_path.empty? %>
-ENV['GEM_HOME'] <%= assignment_operator %> File.expand_path('../..', __FILE__)
-<% else %>
-ENV['GEM_HOME'] <%= assignment_operator %> File.expand_path('../../<%= config.relative_gem_path %>', __FILE__)
-<% end %>
-<% if config.bundler && config.bundler[:gemfile_path] %>
+<% if config.relative_gem_path.empty? -%>
+ENV['GEM_HOME'] <%= config.override_gem_home ? '=' : '||=' %> File.expand_path('../..', __FILE__)
+<% else -%>
+ENV['GEM_HOME'] <%= config.override_gem_home ? '=' : '||=' %> File.expand_path('../../<%= config.relative_gem_path %>', __FILE__)
+<% end -%>
+<% if config.override_gem_home -%>
+ENV['GEM_PATH'] = ENV['GEM_HOME']
+<% end -%>
+<% if config.bundler && config.bundler[:gemfile_path] -%>
 ENV['BUNDLE_GEMFILE'] = File.expand_path('../../<%= config.bundler[:gemfile_path] %>', __FILE__)
-<% end %>
+<% end -%>

lib/warbler/templates/rails.erb

@@ -1 +1 @@
-ENV['RAILS_ENV'] ||= '<%=  config.webxml.rails.env %>'
+ENV['RAILS_ENV'] ||= '<%= config.webxml.rails.env %>'

lib/warbler/templates/war.erb

@@ -1,12 +1,17 @@
-<% assignment_operator = config.override_gem_home ? "=" : "||=" %>
 if $servlet_context.nil?
-  ENV['GEM_HOME'] <%= assignment_operator %> File.expand_path(File.join('..', '..', '<%= config.gem_path %>'), __FILE__)
-<% if config.bundler && config.bundler[:gemfile_path] %>
+  ENV['GEM_HOME'] <%= config.override_gem_home ? '=' : '||=' %> File.expand_path(File.join('..', '..', '<%= config.gem_path %>'), __FILE__)
+<% if config.override_gem_home -%>
+  ENV['GEM_PATH'] = ENV['GEM_HOME']
+<% end -%>
+<% if config.bundler && config.bundler[:gemfile_path] -%>
   ENV['BUNDLE_GEMFILE'] ||= File.expand_path(File.join('..', '..', '<%= config.bundler[:gemfile_path] %>'), __FILE__)
-<% end %>
+<% end -%>
 else
-  ENV['GEM_HOME'] <%= assignment_operator %> $servlet_context.getRealPath('<%= config.gem_path %>')
-<% if config.bundler && config.bundler[:gemfile_path] %>
+  ENV['GEM_HOME'] <%= config.override_gem_home ? '=' : '||=' %> $servlet_context.getRealPath('<%= config.gem_path %>')
+<% if config.override_gem_home -%>
+  ENV['GEM_PATH'] = ENV['GEM_HOME']
+<% end -%>
+<% if config.bundler && config.bundler[:gemfile_path] -%>
   ENV['BUNDLE_GEMFILE'] ||= $servlet_context.getRealPath('/<%= config.bundler[:gemfile_path] %>')
-<% end %>
+<% end -%>
 end