Skip to content

Conversation

@daniandtheweb
Copy link
Contributor

@daniandtheweb daniandtheweb commented May 5, 2023

This pull request adds a check in the Makefile in order to set automatically the -lcblas flag on Arch Linux. This change is based on koboldcpp's Makefile so the credit for this should go to @LostRuins.

This commit is a port of a detection method used in koboldcpp's Makefile in order to automatically set the -lcblas option on Arch Linux
Copy link

@SergeAx SergeAx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is legit. Here's the original commit: LostRuins@7ba36c2

daniandtheweb and others added 2 commits May 5, 2023 22:51
Co-authored-by: Pavol Rusnak <[email protected]>
Co-authored-by: Pavol Rusnak <[email protected]>
Copy link
Contributor

@prusnak prusnak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also test on Arch Linux whether my suggestions work. I don't have one laying around.

@daniandtheweb
Copy link
Contributor Author

daniandtheweb commented May 5, 2023

Like this it works perfectly on Arch Linux. Sorry for the mess with the commits but it's my first time editing a Makefile.

@prusnak prusnak merged commit 173d0e6 into ggml-org:master May 5, 2023
@prusnak
Copy link
Contributor

prusnak commented May 5, 2023

Sorry for the mess with the commits but it's my first time editing a Makefile.

No worries, you did great and I really like the final form of the submission! 🎉

@LostRuins
Copy link
Collaborator

Nice 👍

KerfuffleV2 pushed a commit to KerfuffleV2/llama.cpp that referenced this pull request May 6, 2023
This commit is a port of a detection method used in koboldcpp's Makefile in order to automatically set the -lcblas option on Arch Linux
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.

4 participants