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

coder 2.1.4 #96

Merged
merged 9 commits into from
Aug 29, 2023
Merged

coder 2.1.4 #96

merged 9 commits into from
Aug 29, 2023

Conversation

aslilac
Copy link
Member

@aslilac aslilac commented Aug 29, 2023

  • Rename old formulae to all point at "coder@1" (since they've been identical for a long time anyway)
  • Refactor coder@1 formula to simplify version updating
  • Introduce new coder formula for installing full v2 binaries
  • Add a script for updating the coder formula
  • Setup tests to use the "official" brew testing pattern
    • Includes all the nice things you'd expect like code formatting, hash validation, running brew tests, etc.

@aslilac aslilac marked this pull request as ready for review August 29, 2023 18:11
@aslilac aslilac requested a review from bpmct August 29, 2023 18:11
Copy link
Member

@deansheather deansheather left a comment

Choose a reason for hiding this comment

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

Looks good, Just to clarify with the renaming, anyone with the old CLI installed will keep the old CLI installed until they uninstall and install the new one?

Comment on lines +15 to +16
url "https://github.com/coder/coder/releases/download/v#{version}/coder_#{version}_linux_amd64.tar.gz"
sha256 "41666bbe3afacd153fbe6c1a2d908bb4fb7a88e821205cb7136a0bad2d1cd6dc"
Copy link
Member

Choose a reason for hiding this comment

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

We have armv7, arm64 and amd64 for Linux so it'd be nice to do the Hardware::CPU checks for those too.

Copy link
Member Author

Choose a reason for hiding this comment

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

Homebrew only supports x64 linux, unfortunately. It won't install or run on anything else. :\ If they ever add that platform support tho, we should 100% match it.

@aslilac
Copy link
Member Author

aslilac commented Aug 29, 2023

Just to clarify with the renaming, anyone with the old CLI installed will keep the old CLI installed until they uninstall and install the new one?

Correct. The way Homebrew handles this internally is actually super clever. :)

@aslilac aslilac merged commit 508bff6 into tap-refresh Aug 29, 2023
2 checks passed
@aslilac aslilac deleted the coder-v2 branch August 29, 2023 19:09
@aslilac aslilac restored the coder-v2 branch August 29, 2023 19:10
@aslilac aslilac mentioned this pull request Aug 29, 2023
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