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 Goreleaser to build releases #113

Merged
merged 3 commits into from
Apr 12, 2024
Merged

Add Goreleaser to build releases #113

merged 3 commits into from
Apr 12, 2024

Conversation

dehanj
Copy link
Member

@dehanj dehanj commented Apr 2, 2024

The files needed to build the packages I have put in the system/ folder. Some of these are duplicates from the debian folder, this is intentional since I think we can remove the ´Debian`folder going forward.

The only noticeable difference between our "old" Debian package and these are that the tkey-ssh-agent.1 was gzip. It is not here.

Needs to be verified:

  •  Ubuntu (deb-pkg)
  • Alpine installation
  • Arch Linux
  • RPM
  • Windows binaries
  • macOS signature
  • macOS binaries

@dehanj dehanj linked an issue Apr 2, 2024 that may be closed by this pull request
@dehanj dehanj requested a review from mchack-work April 2, 2024 12:40
@dehanj dehanj marked this pull request as draft April 2, 2024 13:02
@dehanj dehanj marked this pull request as ready for review April 2, 2024 14:13
@dehanj
Copy link
Member Author

dehanj commented Apr 2, 2024

The generated windows binaries are yet to be verified. But it is ready for review and testing.

Things to look for;

  • special cases for any of Linux packages.
  • How do we verify all the Linux packages?
  • In the archives we today include the man-page, should we include something more? Like the copyright? Something else?

.goreleaser.yaml Outdated Show resolved Hide resolved
@dehanj
Copy link
Member Author

dehanj commented Apr 5, 2024

I have continued to verify:

  • Signature on our Darwin binary seems OK.
  • Alpine installation works. Since we don't sign the packages at the moment one needs to pass --allow-untrusted.
  • We are shipping unnecessary files for alpine, most likely.
  • Man pages work as well on Alpine, if one installs a mandoc for example.

Should we delete the "old" debian folder in this PR directly? Wont be used anymore.

@dehanj
Copy link
Member Author

dehanj commented Apr 8, 2024

Windows binaries (including tray) checked on windows.

@dehanj
Copy link
Member Author

dehanj commented Apr 10, 2024

Wrapped the pre-hook command in "sh -c " is the answer to make it run properly. Added a commit that fixes it.

@mchack-work mchack-work force-pushed the add_goreleaser branch 4 times, most recently from 0432a88 to 6b115a6 Compare April 12, 2024 07:30
dehanj and others added 3 commits April 12, 2024 13:31
Since we now build .deb packages with goreleaser we remove the debian
directory.
Let's use -buildvcs=false

If the Git directory even has some untracked files the binary will be
different with buildvcs tracking. This will probably shoot us in the
foot again and again.
- Use checkout@v4
- Check för SPDX tags early
Copy link
Member

@mchack-work mchack-work left a comment

Choose a reason for hiding this comment

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

LGTM.

@mchack-work mchack-work merged commit ea8dc6e into main Apr 12, 2024
5 checks passed
@mchack-work mchack-work deleted the add_goreleaser branch April 12, 2024 11:35
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.

Use Goreleaser to build releases
2 participants