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

clib: add clib.json #954

Merged
merged 1 commit into from
Jul 7, 2024
Merged

clib: add clib.json #954

merged 1 commit into from
Jul 7, 2024

Conversation

mofosyne
Copy link
Contributor

@mofosyne mofosyne commented Jun 22, 2024

Adding support for https://github.com/clibs/clib a c package manager

When done you may want to add to https://github.com/clibs/clib/wiki/Packages#hashing so it is discoverable

@Cyan4973
Copy link
Owner

Can the clib.json file be in another place than root directory?

@mofosyne
Copy link
Contributor Author

inspecting the clib code, unfortunately it assumes that it's located in the root of the repository e.g. searching for manifest_names in clib

It's just this one file at least.

@mofosyne
Copy link
Contributor Author

mofosyne commented Jul 6, 2024

@Cyan4973 is this ready to merge? I'm hoping to point ggerganov/llama.cpp#8048 to your repo instead.

@Cyan4973 Cyan4973 merged commit d5fe4f5 into Cyan4973:dev Jul 7, 2024
62 checks passed
@mofosyne mofosyne deleted the clibs branch July 7, 2024 20:29
mofosyne added a commit to ggerganov/llama.cpp that referenced this pull request Jul 15, 2024
Convinced Cyan4973 to add clib.json directly to his repo, so can now point the clib package directly to him now. Previously pointed to my fork with the clib.json package metadata

Cyan4973/xxHash#954
ggerganov pushed a commit to ggerganov/llama.cpp that referenced this pull request Jul 16, 2024
* Update clib.json to point to Cyan4973 original xxhash

Convinced Cyan4973 to add clib.json directly to his repo, so can now point the clib package directly to him now. Previously pointed to my fork with the clib.json package metadata

Cyan4973/xxHash#954

* gguf-hash: readme update to point to Cyan4973 xxHash repo [no ci]
@mofosyne
Copy link
Contributor Author

@Cyan4973 thanks. I've pointed the package in llama.cpp's llama-gguf-hash to you now :)

arthw pushed a commit to arthw/llama.cpp that referenced this pull request Jul 27, 2024
…nov#8491)

* Update clib.json to point to Cyan4973 original xxhash

Convinced Cyan4973 to add clib.json directly to his repo, so can now point the clib package directly to him now. Previously pointed to my fork with the clib.json package metadata

Cyan4973/xxHash#954

* gguf-hash: readme update to point to Cyan4973 xxHash repo [no ci]
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.

2 participants