We no longer want to not load cache.inc as it holds the cache interface and the wrapper functions. To my best knowledge, memcache is the first alternative cache backend to be ported to the D7 cache mechanism so we only found this now.

CommentFileSizeAuthor
#4 cache_inc_mandatory.patch1.25 KBchx
#2 cache_inc_mandatory.patch1.25 KBchx
cache_inc_mandatory.patch731 byteschx

Comments

Status: Reviewed & tested by the community » Needs work

The last submitted patch failed testing.

chx’s picture

chx commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new cache_inc_mandatory.patch1.25 KB

Why on earth did I load that variable with its default value in install.php?

dries’s picture

dries commented

This makes sense. Wondering if we should rename cache_inc to cache_backends or something.

chx’s picture

chx commented
StatusFileSize
new cache_inc_mandatory.patch1.25 KB

Certainly.

dries’s picture

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

Committed to CVS HEAD. Thanks!

moshe weitzman’s picture

moshe weitzman commented

I'm not sure if this patch is to blame, but I am getting a white screen when trying to upgrade from d6. When we boostrap throiugh tbe variables phase, we try to query cache_bootstrap table but that has not been created yet. Are we missing a DrupalFakeCache for update.php?

chx’s picture

chx commented

One thing for sure: cache_inc is not used in update.php. We might need to precreate the cache_bootstrap table but that'd be a diff (but important) issue.

Status: Fixed » Closed (fixed)

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