BLT-5206: Remove cache.php, no longer needed for drush about blt

[danepowell]

Thanks. I haven't thought about this file in a while but if it's no longer needed I'm happy to remove it: #4665

I couldn't find it referenced anywhere else, do you know if it's part of an include or require anywhere?

[bkosborne]

Could be used by BLT plugins. The one for sure that's using it is the ACSF plugin

[bkosborne]

@danepowell Might want to first get this removed in the ACSF plugin. As is, if people upgrade BLT, their deployments will break as the ACSF plugin tries to use this file.

[danepowell]

@bkosborne could you please open a PR for blt-acsf? Otherwise I'll need to revert this. I don't use, maintain, or test that plugin. I can only review community PRs.

[anairamzap-mobomo]

Hi, just a note to say that indeed removing the cache.php file breaks acsf deployments using a db-update script as provided by the Acquia docs: https://docs.acquia.com/site-factory/extend/hooks/post-site-update/

I can see the code itself in the repo https://github.com/acquia/blt-acsf/blob/master/factory-hooks/db-update/db-update.sh was updated, but it would be great to also update the documentation.

Just posting this comment here in case it helps someone else.