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

Use tar.gz for Mac and Linux #2351

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

Conversation

samrocketman
Copy link

@samrocketman samrocketman commented Feb 13, 2025

Changes

Changes GitHub releases to handle Mac and Linux binaries slightly differently than Windows.

Operating System Archive Format
Mac OS tar.gz
Linux tar.gz
Windows zip

Tests

I tested with the following commands.

$ goreleaser release --snapshot

$ ls -1 dist/*.{zip,tar.gz}
dist/databricks_cli_darwin_amd64.tar.gz
dist/databricks_cli_darwin_arm64.tar.gz
dist/databricks_cli_linux_amd64.tar.gz
dist/databricks_cli_linux_arm64.tar.gz
dist/databricks_cli_windows_amd64.zip
dist/databricks_cli_windows_arm64.zip

Related issues

Closes #1296

Changes
-------

Changes GitHub releases to handle Mac and Linux binaries slightly differently than Windows.

| Operating System | Archive Format |
| ---------------- | -------------- |
| Mac OS           | `tar.gz`       |
| Linux            | `tar.gz`       |
| Windows          | `zip`          |

Tests
-----

I tested with the following commands.

```bash
$ goreleaser release --snapshot

$ ls -1 dist/*.{zip,tar.gz}
dist/databricks_cli_darwin_amd64.tar.gz
dist/databricks_cli_darwin_arm64.tar.gz
dist/databricks_cli_linux_amd64.tar.gz
dist/databricks_cli_linux_arm64.tar.gz
dist/databricks_cli_windows_amd64.zip
dist/databricks_cli_windows_arm64.zip
```

Related issues
--------------

Closes databricks#1296

Co-authored-by: Sam Gleske <[email protected]>
Copy link

An authorized user can trigger integration tests manually by following the instructions below:

Trigger:
go/deco-tests-run/cli

Inputs:

  • PR number: 2351
  • Commit SHA: 5f7539379ac6517662d9bd0997a807c9a43416a1

Checks will be approved automatically on success.

@pietern
Copy link
Contributor

pietern commented Feb 13, 2025

Thanks for the PR. We can't accept this as is because changing the archive format is a breaking change for automation.

We'll consider adding .tar.gz as an additional format.

@sgleske-ias
Copy link

@pietern goreleases supports shipping multiple archive formats so I can add overrides for Mac and Linux to provide both .tar.gz and .zip; would that be acceptable?

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.

Provide .gz or tar.gz binary for streaming download and extracting
3 participants