Skip to content

Commit

Permalink
Merge pull request #3266 from NickLaMuro/leak_fix_to_s_autoload_paths
Browse files Browse the repository at this point in the history
Fix memory leak with ruby/require/autoload_paths
(cherry picked from commit 62653f6)

https://bugzilla.redhat.com/show_bug.cgi?id=1536672
  • Loading branch information
Fryguy authored and simaishi committed Jan 19, 2018
1 parent d410aaf commit 6862353
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/manageiq/ui/classic/engine.rb
Expand Up @@ -26,8 +26,8 @@ module ManageIQ
module UI
module Classic
class Engine < ::Rails::Engine
config.autoload_paths << File.expand_path(File.join(root, 'app', 'controllers', 'mixins'), __FILE__)
config.autoload_paths << File.expand_path(File.join(root, 'lib'), __FILE__)
config.autoload_paths << root.join('app', 'controllers', 'mixins').to_s
config.autoload_paths << root.join('lib').to_s

initializer 'manageiq-ui-classic.assets.precompile' do |app|
app.config.assets.precompile << proc do |filename, path|
Expand Down

0 comments on commit 6862353

Please sign in to comment.