Commit 2024-03-19 03:35 efca0243

View on Github →

fix(Cache): do not read lake-manifest.json at import-time (#11492) Previously this file was read four times at runtime, and once for each file at compile time. Now it is read only once, when lake exe cache is run. initialize is not a substitute for correctly passing global state through a program; my impression is that it is intended for language- rather than program-level initialization. The performance impact is negligible, but the new code is more predictable. Tested on mathlib, and on a package created using lake new foo math, pointed at this mathlib branch. lake exe cache get works correctly (leaves lake build with nothing to do) in both cases.

Estimated changes