resourceloader: Disable filesystem cache for Less
authorTimo Tijhof <krinklemail@gmail.com>
Thu, 4 Aug 2016 21:47:57 +0000 (14:47 -0700)
committerTimo Tijhof <krinklemail@gmail.com>
Thu, 4 Aug 2016 21:51:55 +0000 (14:51 -0700)
commitc2a41baee5cf422489a7b8a8d744e0a0158bbdc3
treea3de1bf7d83aa8dc9a2b2595302d1a6066e03575
parentd181ba1a9b00e7b7e418b82e0b0def3cc184e48d
resourceloader: Disable filesystem cache for Less

Follows-up ea862efa7, which enables use of Less_Cache as part
of the switch from leafo/lessphp to oyejorge/less.php.

Since then 12afb3607 introduced a better caching method in
FileModule::compileLessFile(), which:

* Uses APC where possible, with fallback to main objectcache.
* Verifies both the input file and the indirectly used files have
  not changed before using the cache. The built-in cache system
  only seems to check the root file.

As such, we're incurring the overhead of generating them and storing
on-disk, but raraly ever making use of them. (And even if we do
use it when the APC object first expires, it's risky due to the
lack of indirect file validation).

Bug: T121519
Change-Id: I6e3c66f86a536463488697efc166fb704be0715c
includes/resourceloader/ResourceLoader.php