Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@spiffcs
Copy link
Contributor

@spiffcs spiffcs commented Oct 19, 2021

Add arm64 goarch as a release binary.

Resolves #362

Signed-off-by: Christopher Angelo Phillips [email protected]

Signed-off-by: Christopher Angelo Phillips <[email protected]>
@spiffcs spiffcs requested a review from a team October 19, 2021 16:00
Comment on lines +32 to +33
- binary: grype
id: grype-arm64
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we'd want to make sure we're taking the same codesigning steps for this darwin/arm64 binary that we do today for the darwin/amd64 binary — let me know if you want to chat through this!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's my bad @luhring I forgot to update the goos to linux for amd64. I don't think there is any other signing we need to include if this is a linux release.

I added build under archives so zip artifacts are included for this new build ID.

Let me know if you need anything else here.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah gotcha! Good clarification. I think the only "signing" would be that this new binary gets its checksum added to the ...checksums.txt file, and that file itself gets signed. Does that sound right?

Copy link
Contributor Author

@spiffcs spiffcs Oct 20, 2021

Choose a reason for hiding this comment

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

Right now our checksums file output looks like this:

38c9917da5f43702770307095e063749f7d1b92222e840d46973acd029b55c4c  grype_0.23.0_linux_amd64.rpm
4e04801f614981b1e7c2d8bb32be53c0e2679bfd7b64a30315515f23133cc9a9  grype_0.23.0_linux_amd64.deb
d277030c12fdfe1a43503c95a34fb1e1556f1dba8d42458abb2b542a201828ba  grype_0.23.0_linux_amd64.tar.gz

Adding the block in this PR to the builds section would mean the new build should be appended. Everything but the zip artifact is included here. I don't think there is any other code to change.

- 'sed "/.*\.zip/d" ${artifact} > tmpfile && mv tmpfile ${artifact} && gpg --output ${signature} --detach-sign ${artifact}'

Copy link
Contributor

@luhring luhring left a comment

Choose a reason for hiding this comment

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

This looks good, and I'll assume you've successfully tested it out. This is a great addition! 👏

nit: It might help to add "Linux" to the PR title to make it clear what we mean by adding an arm64 build

@spiffcs spiffcs changed the title 362-add binary for arm64 to release process 362 add binary for linux arm64 to release process Oct 20, 2021
Signed-off-by: spiffcs <[email protected]>
Signed-off-by: Christopher Angelo Phillips <[email protected]>
@spiffcs spiffcs merged commit 441a643 into main Oct 20, 2021
@spiffcs spiffcs deleted the 362-arm64-release branch October 20, 2021 21:11
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.

Add a release for linux/arm64

2 participants