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.

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

Build static lib #772

Closed
wants to merge 1 commit into from
Closed

Build static lib #772

wants to merge 1 commit into from

Conversation

iacore
Copy link
Contributor

@iacore iacore commented Apr 5, 2023

make now creates a static library too.

I haven't modified CMake-related stuff. Not sure if that is needed.

@prusnak
Copy link
Collaborator

prusnak commented Apr 5, 2023

It's a good practice to say WHY the change is required. So what is your motivation for the change? What is your usecase?

@iacore
Copy link
Contributor Author

iacore commented Apr 5, 2023

This patch by itself allows this repo to be used as a C++ library.

In the future there might be C bindings to libllama, making llama.cpp available to many other languages.

@ggerganov
Copy link
Member

@iacore
It has the same problem as the other PR - the common stuff should not be part of llama library.
There are only utility functions that are specific for the examples and 3rd party projects should not rely on those

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants