Improve Memcache as installation profile friendly

This is exactly the issue I am having. Error on the install.php page

Requirements problem
The following error must be resolved before you can continue the installation process:
Memcache admin requires that the "cache_inc" $conf variable be set to include either memcache.inc or memcache.db.inc. Please read the installation instructions. (Currently using Memcache admin Memcache not included)

Please check the error messages and try again.

Hopefully a fix is out there for this (that can void the temporary removal of the memcache module)

Using drupal 6.20, memcache module 1.7 on Ubuntu 10.10

Thanks
Sean

As refer to memcache CVS HEAD for Drupal 7.x, no memcache_admin.install exists and so all running smooth without any problem (http://drupalcode.org/viewvc/drupal/contributions/modules/memcache/memca...). It is working fine with my Drupal 7.x installation profile development.

I would like to suggest "remove" the memcache_admin.install in order to solve this problem. Any idea?

I clone the similar checking style from getid3 and colorbox, so update the test case only for $phase == 'runtime'.

Patch via memcache DRUPAL-6--1 CVS, and successfully pass the installation profile checking.

This issue is still exists within lastest memcache-6.x-1.8 and so block the integration with our PantaRei Siren installation profile (http://drupal.org/project/pantarei_siren). Therefore we need to maintain the synchronize of patch from #3 manually.

May someone provide a kindly review of this issue so we can get fixed within official version?

I currently applied the patch against the 6.x-1.8 version and all reported problems vanished. So I guess we can live with the fact, that the checking is done only during runtime and not during install as the Memcache admin module only provides information about memcache and does not really require a memcache server to work.

Hope to see this in an official release soon, as this is really a big problem for install profiles.

Cheers

hctom

Thanks! Committed this to latest dev.

No memcache install in 7.x.