feat(ci): add caching for CMake FetchContent in CI workflow#3448
feat(ci): add caching for CMake FetchContent in CI workflow#3448jihuayu wants to merge 3 commits intoapache:unstablefrom
Conversation
PragmaTwice
left a comment
There was a problem hiding this comment.
You can use DEPS_FETCH_DIR instead of caching these _deps directories (which may bring some chaos).
…or improved build process
|
@PragmaTwice Thanks for the reminder. I’ve made the changes. |
|
I don't think we need this. They are useless and introduce complexity.
I think we'd better to choose the simplest way. |
This variable is just for readability; otherwise, we’d be stuck facing a very long cache key. As I understand it, the Lua and jemalloc options must be cached separately because they pull different source codes. Is that correct? |
Nope. The dep dir can be shared. The filename is different for different deps. |



Due to the current instability of GitHub services, dependency download failures occur frequently during CI runs. I add FetchContent caching to prevent these network issues from failing the CI.