From Drupal Status report after updating the 7.x.-1.8.

Warning
Memcache 3.0.8 (more information)
There is a problem with your memcache configuration. Please review
sites/all/modules/memcache/README.txt for help resolving the following
issue:

PECL Memcache version 3.0.8 is unsupported. Please update to 3.0.6 or newer.

Seems to be a typo in memcache.install with changes made up to 7.x-1.8.

@@ -269,7 +269,7 @@ function _memcache_pecl_version_valid() {
// it will require a replacement for version_compare().
$full_version = phpversion('memcache');
$version = implode('.', array_slice(explode('.', $full_version), 0, 3));
- if (version_compare(phpversion(), '7'. '>=')) {
+ if (version_compare(phpversion(), '7', '>=')) {
return version_compare($version, MEMCACHE_PECL_PHP7_RECOMMENDED, '>=');
}
else {

CommentFileSizeAuthor
#3 memcache-version-typo-fix-3160924-3.patch575 bytessolideogloria

Issue fork memcache-3160924

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

kaosagnt created an issue. See original summary.

kaosagnt’s picture

kaosagnt commented
Title: Drupal complains perl memcache is not correct version » Drupal complains pecl memcache is not correct version
solideogloria’s picture

solideogloria commented
Status: Active » Needs review
StatusFileSize
new memcache-version-typo-fix-3160924-3.patch575 bytes

Consider adding a patch file next time for such a simple issue.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented

I don't see what changes on that line ? Looks like it's just a space at the end of line ?

mmjvb’s picture

mmjvb commented

By replacing '.' with ',' now it is passing three parameters instead of two!

solideogloria’s picture

solideogloria commented

Yep. Consider getting the Git Patch Viewer browser extension to make patch files easier to read.

Jeremy made their first commit to this issue’s fork.

Jeremy opened merge request !7

jeremy’s picture

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

Thanks! Really late to reviewing, looks great.

  • Jeremy committed e1d22ff on 7.x-1.x authored by solideogloria 
    Issue #3160924 by solideogloria, kaosagnt: Drupal complains pecl...
jeremy’s picture

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

Merged, thanks!

Status: Fixed » Closed (fixed)

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