Attached patch shows the version of the APC module on the reports page (as link) and additionally shows a warning if the apc module isn't loaded in PHP.

CommentFileSizeAuthor
#2 1526740-requirements.patch1.09 KBcafuego
apc-requirements.patch1.11 KBcafuego

Comments

cafuego’s picture

cafuego commented
Status: Active » Needs review

For the record, this same patch also applies cleanly to 7.x-1.x.

cafuego’s picture

cafuego commented
StatusFileSize
new 1526740-requirements.patch1.09 KB

.. or rather, it would if I hadn't broken it before making the patch and then fixed it afterward :-P

bfroehle’s picture

bfroehle commented

Looks good, I'll commit this soon.

bfroehle’s picture

bfroehle commented

Is there are reason to have this check at install or update time (instead of just runtime)?

cafuego’s picture

cafuego commented

[edit] My previous comment made no sense :-)

Yeah, I think that running only during runtime is probably a much better idea. No need to break installation just because a non-essential php extension isn't present. We do then need to make certain that no APC functions are called without checking that the extension is there.