Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Updates Package.swift to use ggml as package dependency #1701

Merged
merged 3 commits into from
Jan 3, 2024

Conversation

1-ashraful-islam
Copy link
Contributor

When adding both whisper.cpp and llama.cpp as package dependency in Xcode - we run into "duplicate declarations of symbols" issue as both of these package uses the same source files for ggml.

Adding a swift package declaration to ggml (ggml-org/ggml/pull/674) and importing it as a dependency to both whisper.cpp and llama.cpp solves the issue.

Once the pull request (ggml-org/ggml/pull/674) is merged, merge this pull requests to whisper.cpp that use ggml as dependency.

Related: (llama.cpp/pull/4691)

@ggerganov ggerganov merged commit 993acb5 into ggerganov:master Jan 3, 2024
ggerganov added a commit that referenced this pull request Feb 12, 2024
ggerganov added a commit that referenced this pull request Feb 12, 2024
* Revert "swift : update Package.swift to use ggml as package dependency (#1701)"

This reverts commit 993acb5.

* spm : add ggml.h
jiahansu pushed a commit to WiseSync/whisper.cpp that referenced this pull request Apr 17, 2024
* Revert "swift : update Package.swift to use ggml as package dependency (ggerganov#1701)"

This reverts commit 993acb5.

* spm : add ggml.h
viktor-silakov pushed a commit to viktor-silakov/whisper_node_mic.cpp that referenced this pull request May 11, 2024
…anov#1701)

* updates Package.swift to use ggml as dependency

* cleans up the Package.swift file by removing redundant source files

* updates ggml url src to ggerganov
viktor-silakov pushed a commit to viktor-silakov/whisper_node_mic.cpp that referenced this pull request May 11, 2024
* Revert "swift : update Package.swift to use ggml as package dependency (ggerganov#1701)"

This reverts commit 993acb5.

* spm : add ggml.h
iThalay pushed a commit to iThalay/whisper.cpp that referenced this pull request Sep 23, 2024
…anov#1701)

* updates Package.swift to use ggml as dependency

* cleans up the Package.swift file by removing redundant source files

* updates ggml url src to ggerganov
iThalay pushed a commit to iThalay/whisper.cpp that referenced this pull request Sep 23, 2024
* Revert "swift : update Package.swift to use ggml as package dependency (ggerganov#1701)"

This reverts commit 993acb5.

* spm : add ggml.h
iThalay pushed a commit to iThalay/whisper.cpp that referenced this pull request Sep 23, 2024
* Revert "swift : update Package.swift to use ggml as package dependency (ggerganov#1701)"

This reverts commit 993acb5.

* spm : add ggml.h
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants