-
Notifications
You must be signed in to change notification settings - Fork 4.2k
CCache not found warning persists even when GGML_CCACHE is set to off #3063
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Can you try using |
Yes that works, thanks. Should GGML_CCACHE be reset to ON if someone hasn't set WHISPER_CCACHE, or should the make files be changed to respect the flag? |
I think that |
So we would not want to use something like If I'm thinking properly, this would set GGML_CCACHE to the value of WHISPER_CCACHE, except in those cases in which the user manually passed the flag GGML_CCACHE. Does Whisper itself use CCache at all outside of the GGML usage? While |
I think the idea is that |
I agree that it's better to have the one source of truth. Maybe in retrospect what I'm after is really just clarity in the logs/warnings/errors, since a first-level step with the warning message would be to set GGML_CCACHE to OFF, only for it to not work. Could the warning message be changed to something that points someone to WHISPER_CCACHE directly or hints at checking for a different flag that overrides that setting? (I don't actually know what the relationship between GGML in this repository and the GGML-dedicated repo is, so maybe this is inappropriate to change here.) |
Ah yes, I see your point now. Let me think about this and take a look at what could be done. |
When configuring with
cmake -DGGML_CCACHE=OFF -B build
, I still receive a warningWarning: ccache not found - consider installing it for faster compilation or disable this warning with GGML_CCACHE=OFF
, which seems to come from ggml/src/CMakeLists.txt.I'm not sure why this might be happening, as I'd expect that status message to not be printed since it's within an
IF(GGML_CCACHE…
statement. Do I need to be setting GGML_CCACHE elsewhere as well?The text was updated successfully, but these errors were encountered: