Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Make fails #63

Closed
shaheenkdr opened this issue Oct 18, 2022 · 0 comments
Closed

Make fails #63

shaheenkdr opened this issue Oct 18, 2022 · 0 comments
Labels
build Build related issues

Comments

@shaheenkdr
Copy link

g++ -O3 -std=c++11 -Wall -Wextra -Wno-unused-parameter -Wno-unused-function -pthread -c whisper.cpp whisper.cpp: In function ‘whisper_full_params whisper_full_default_params(whisper_decode_strategy)’: whisper.cpp:2286:17: sorry, unimplemented: non-trivial designated initializers not supported }; ^ whisper.cpp:2313:17: sorry, unimplemented: non-trivial designated initializers not supported }; ^ Makefile:74: recipe for target 'whisper.o' failed make: *** [whisper.o] Error 1

g++ (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
@ggerganov ggerganov added the build Build related issues label Oct 18, 2022
ggerganov added a commit that referenced this issue Oct 18, 2022
@ggerganov
ref #57, #62, #63 : remove unions in C-api + remove designated initia…
e30cf83 
…lizers

We are not ready for designated initializers - many compilers do not
support this C++ feature yet, so removing it's non-trivial usages.
anandijain pushed a commit to anandijain/whisper.cpp that referenced this issue Apr 28, 2023
@ggerganov
ref ggerganov#57, ggerganov#62, ggerganov#63 : remove unions in C-api…
7a3d04b 
… + remove designated initializers

We are not ready for designated initializers - many compilers do not
support this C++ feature yet, so removing it's non-trivial usages.
jacobwu-b pushed a commit to jacobwu-b/Transcriptify-by-whisper.cpp that referenced this issue Oct 24, 2023
@ggerganov
ref ggerganov#57, ggerganov#62, ggerganov#63 : remove unions in C-api…
025cb8f 
… + remove designated initializers

We are not ready for designated initializers - many compilers do not
support this C++ feature yet, so removing it's non-trivial usages.
kultivator-consulting pushed a commit to KultivatorConsulting/whisper.cpp that referenced this issue Feb 12, 2024
@tazz4843
Merge pull request ggerganov#63 from xandkar/sk/update-readme-example
9cffd74 
Fix/update example in the README
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
build Build related issues
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ggerganov @shaheenkdr