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.

Sign up for GitHub

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

Jump to bottom

Include ARCH in asset name #219

Merged

yetone merged 1 commit into yetone:main from rkk1995:main

Aug 25, 2024

Contributor

rkk1995 commented Aug 25, 2024

Currently, the makefile tries to cURL an asset that is no longer part of the latest releases.

For example, previously the release had the file tiktoken_core-linux-luajit.so.
Now, the CPU arch is part of the release asset. See https://github.com/gptlang/lua-tiktoken/releases/tag/v0.2.2

This change adds ARCH so that we fetch an existing asset.

I tested this by telling lazy to use my own branch and the file downloaded is correct

➜  build git:(84fa8df) ls -lh
Permissions Size User Date Modified Name
.rw-r--r--  3.1M rkk  25 Aug 15:50  tiktoken_core.so


          Include ARCH in asset name

84fa8df

Owner

yetone commented Aug 25, 2024

LGTM

yetone merged commit 5159aee into yetone:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet