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

Add homebrew formula for published testing binaries #4

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

anuraaga
Copy link
Collaborator

@anuraaga anuraaga commented Oct 3, 2024

After this is merged, it should be possible to install with

brew tap elastic/ollama https://github.com/elastic/ollama # 2nd arg needed since this repo's name isn't homebrew-ollama
brew install ollama-elastic
ollama-elastic --version

I checked the tap locally on both macos and linux/amd64 - homebrew doesn't seem to support linux/arm64.

Followed this for inspiration, especially the use of prebuilt binaries. It's not idiomatic in homebrew proper to do that, but given the build is quite heavy I think this is simpler than build from source (spent a couple hours today getting ollama to build again after having updated macos to sequoia...).

https://github.com/ymtdzzz/homebrew-tap/blob/main/otel-tui.rb

end
end
end
on_arm do
Copy link
Collaborator

Choose a reason for hiding this comment

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

a little confused per "homebrew doesn't seem to support linux/arm64." should we still build?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Figured it doesn't hurt to keep it here in case brew adds support out of band later. If it's confusing, can remove it. The build won't be faster unfortunately since it's the Darwin artifact that takes quite a while.

Copy link
Collaborator

Choose a reason for hiding this comment

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

makes sense

@anuraaga anuraaga merged commit 809ec99 into elastic:main Oct 4, 2024
13 checks passed
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