-
Notifications
You must be signed in to change notification settings - Fork 385
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
Add musl Compilation #1100
base: master
Are you sure you want to change the base?
Add musl Compilation #1100
Conversation
Hi @Coopaguard, thanks for this PR. I don't personally know much at all about Musel or Alpine, could you explain a bit about what they're for? From a bit of research it looks like this would improve compatibility with Docker? |
Important Note To Reviewers: BEFORE MERGING do a binary update that includes the new musl binaries in deps.zip! |
Hi @martindevans , I don't know how to do so. Can't run the update Binaries action manually, |
Alpine Linux uses the musl C standard library instead of glibc, which is used by many other Linux distributions. Here's why it is often necessary to compile libraries with musl to use them on Alpine Linux. Net8, Alpine & Docker So yes this should allow us to create some docker images of LLama.WebAPI ou Lama.Web |
Really sorry, I should have been clearer there! I've updated the comment to clarify - that was a note to people reviewing/merging to do that before merging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your work putting this together! I'll include this when I'm doing the next binary update in the coming weeks, and it should be included in the next version of LLamaSharp.
Notes:
Vulkan & Cuda will not be supported for this time