There is wrong documentation if using the code from README.txt

$conf['cache_backends'] = array('sites/all/modules/varnish/varnish.cache.inc');

is overriding all backends. So to use varnish with memcache there should read:

$conf['cache_backends'][] = 'sites/all/modules/varnish/varnish.cache.inc';
//add memcache as cache_backend
$conf['cache_backends'][] = 'sites/all/modules/memcache/memcache.inc';
CommentFileSizeAuthor
#1 varnish-readme-1525316-1.patch606 bytesmgifford
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mgifford’s picture

Issue summary: View changes
FileSize
606 bytes

Patch included.

plopesc’s picture

Title: Documentation to work with memcache » Documentation to work other modules that provide alternative cache_backends
Version: 7.x-1.0-beta1 » 7.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

This is an easy doc improvement that can help so many people who do not pay attention to the code and just copy and paste the README file info, like me :)

I experienced this bug with a custom module that provides a cache bachend that stop working after using this. This patch will provide some doc that will work on every drupal site.

These changes should be applied to project page too.

Thank you!

  • MiSc committed c13116f on 7.x-1.x authored by mgifford
    Issue #1525316 by wroxbox: Documentation to work other modules that...
MiSc’s picture

Status: Reviewed & tested by the community » Fixed

Ok, committed to latest dev.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.