-
Notifications
You must be signed in to change notification settings - Fork 11.3k
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
fix: add POSIX functionality for Linux compilation #51
Conversation
Can you add a short comment why this is required? (I assume some functions are not recognized but not obvious which ones) |
Sure! |
I meant into the source code just above the Btw the issue #54 mentions |
Which OS and which GCC version do you use? For me the master compiles just fine on Debian 11 with GCC 10.2.1 (even without the proposed define). |
CentOS7 with GCC 10.2.0 |
This flags has been discussed in |
Now thinking more about this, probably the cleanest option is to add compilation flags to the build system like suggested in ggerganov/whisper.cpp#37 + ggerganov/whisper.cpp#576
We can probably set this for all compilers on all OSes, because either a compiler understands the flag and sets the value to the supported level or the flag is ignored. @valentynbez can you confirm this fixed the issue for you on CentOS 7? |
Small fix to compile binaries properly on Linux:
CLOCK_MONOTONIC
inggml.c