Permalink
Browse files

Merge commit 'e792d4ab70448f79142fdf492390682ff5ea6398'

  • Loading branch information...
2 parents 37599d1 + e792d4a commit 9c2657aa96d3a8f19602ded748e558b0531c1132 rick committed May 6, 2008
Showing with 2 additions and 1 deletion.
  1. +2 −1 railties/lib/rails/plugin/locator.rb
@@ -78,7 +78,8 @@ def locate_plugins_under(base_path)
# a <tt>rails/init.rb</tt> file.
class GemLocator < Locator
def plugins
- specs = Gem.loaded_specs.values.select do |spec|
+ specs = initializer.configuration.gems.map(&:specification)
@drnic

drnic May 12, 2008

Contributor

I think this commit is causing the following error – http://pastie.caboo.se/pastes/195002 (pastied by mattly, but it matches what I was seeing in production)

+ specs + Gem.loaded_specs.values.select do |spec|
spec.loaded_from && # prune stubs
File.exist?(File.join(spec.full_gem_path, "rails", "init.rb"))
end

4 comments on commit 9c2657a

and i had worked around that error by installing gems in my system repo instead of vendor/gems

Contributor

chuyeow replied May 12, 2008

Yup I can verify that this is happening for me too (see http://rails.lighthouseapp.com/projects/8994/tickets/122). Can a Rails committer re-open that ticket and possibly consider reverting this commit please? Thanks!

Contributor

chuyeow replied May 12, 2008

Oh yeah and my workaround was to stop using config.gem – I simply reverted to the bit of code I had to load gems in the vendor/gems directory that I was using prior to switching to config.gem.

to be fair — it actually is using my gems in vendor/gems, BUT it requires that they’re also installed in the system repo. I know this because i have a unpack of a gem that breaks on edge rails unpacked into vendor/gems and it actually uses that. but it complains if i don’t have it installed normally too.

Please sign in to comment.