The Drupal 7.x-1.0 INSTALLATION.txt refers to a number of outdated steps, some of which conflict with the steps given in README.txt. This has caused some confusion among colleagues and customers alike. The biggest issue is step 7's recommended changes to settings.php which are not longer valid in D7. Lesser evils are the references to installing an old version of PECL which is no longer supported.

Since README.txt already points to an article on installation of the daemon and PECL extension, can we delete INSTALLATION.txt for the 7.x-1.x branch?

Patch attached.

CommentFileSizeAuthor
memcache-rm-installation-txt.patch4.04 KBjonathan webb

Comments

schnitzel’s picture

schnitzel commented

+1 was also veeery confused by the two files.

pbull’s picture

pbull commented
Status: Needs review » Reviewed & tested by the community

Agreed that installation instructions shouldn't exist in two places (and that docs should be consistent and correct).

Patch succeeded against 7.x-1.x branch.

rickvug’s picture

rickvug commented

Agreed. This led to confusion for our team as well.

patcon’s picture

patcon commented

haha Good lord, I just wasted half a day troubleshooting this exact same thing (due to a silent drush failure in our deployment process)!

Please please please just link the docs in an obvious way to the version-controlled docs :)

patcon’s picture

patcon commented

Funny... just realizing that it was actually the INSTALLATION.txt docs that actually worked for us...

jeremy’s picture

jeremy commented
Status: Reviewed & tested by the community » Fixed

Thanks! It didn't apply cleanly anymore, but I manually fixed that and did a little more cleanup. Committed:
http://drupalcode.org/project/memcache.git/commit/d341d88

jeremy’s picture

jeremy commented
Status: Fixed » Reviewed & tested by the community

Whoops; wrong issue.

jeremy’s picture

jeremy commented
Status: Reviewed & tested by the community » Fixed

Here's the correct commit -- thanks:
http://drupalcode.org/project/memcache.git/commit/b8f5c59

Status: Fixed » Closed (fixed)

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