Skip to content

Commit

Permalink
Fix misleading advice to add 'memcache' to Gemfile
Browse files Browse the repository at this point in the history
[#5539 state:committed]

Commit 5714438 removed the hard-coded dependency on the memcache-client
gem, and added this warning advising people to install it if needed. The
problem is, however, that if people follow the advice literally and install
the 'memcache' gem, they will wind up with a completely different thing,
which is not API compatible with the memcache-client gem and which Rails
can't work with.

So, be explicit and tell users to install the 'memcache-client' gem.

Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
  • Loading branch information
wincent authored and spastorino committed Oct 10, 2010
1 parent 61fc0f0 commit 3a6acb6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion activesupport/lib/active_support/cache/mem_cache_store.rb
@@ -1,7 +1,7 @@
begin
require 'memcache'
rescue LoadError => e
$stderr.puts "You don't have memcache installed in your application. Please add it to your Gemfile and run bundle install"
$stderr.puts "You don't have memcache-client installed in your application. Please add it to your Gemfile and run bundle install"
raise e
end
require 'digest/md5'
Expand Down

0 comments on commit 3a6acb6

Please sign in to comment.