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 linux support #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarkIannucci
Copy link

@MarkIannucci MarkIannucci commented Nov 12, 2023

Hi,

I noticed that these formulae only support MacOS. I use Linux. I tested this out on ubuntu-22.04 and it works like a charm. I tried doing something similar with cockroach and unfortunately it wasn't so easy because of the extra stuff that cockroach does for GEOS support. It'd be great if there was Linux support for all of the formulae in this repo. I'm happy to open an issue to track that need if you'd like.

Thanks!
Mark

@rail rail self-requested a review December 12, 2023 00:29
@@ -5,6 +5,10 @@ class CockroachSql < Formula
desc "Distributed SQL database shell"
homepage "https://www.cockroachlabs.com"
version "23.1.11"
on_linux do
url "https://binaries.cockroachdb.com/cockroach-sql-v23.1.11.linux-amd64.tgz"
sha256 "8a42076aa8d4448820eb47d9816d08f110d830b263e7edfab1e57f354e6817bb"
Copy link
Member

Choose a reason for hiding this comment

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

This file is generated, see the top comment.
In order to add Linux, you need to edit https://github.com/cockroachdb/homebrew-tap/blob/master/release/main.go to include the new platform.
Also, you proably want to separate amd64 and arm64 architectures. And the file needs a bit of refactoring to become more readable with so many platforms.

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