Skip to content

Conversation

@AndreMiras
Copy link

Description

Make static builds and upload them as artifacts.
This makes it possible to download the binary produced for each PR. In a follow up PR we could also automate binary upload upon release as well as enabling cross compilation for different arch and OS.

Note that we use musl-gcc as it allows to statically link a libc implementation while CGO_ENABLED=0 only was leading to the error below:

ethermint@v0.22.0-sdk50-1/app/ante/eip712.go:293:36: undefined: secp256k1.RecoverPubkey
ethermint@v0.22.0-sdk50-1/app/ante/eip712.go:319:17: undefined: secp256k1.VerifySignature

The statically linked binary is 100M as of today.

Also drop get-diff-action as it's unmaintained and was misbehaving on the rebasing and squashing workflow, refs #139

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • targeted the correct branch (see PR Targeting)
  • provided a link to the relevant issue or specification
  • included the necessary unit and integration tests
  • reviewed "Files changed" and left comments if necessary

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

  • confirmed all author checklist items have been addressed
  • reviewed state machine logic
  • reviewed API design and naming
  • reviewed documentation is accurate
  • reviewed tests and test coverage
  • manually tested (if applicable)

Make static builds and upload them as artifacts.
This makes it possible to download the binary produced for each PR.
In a follow up PR we could also automate binary upload upon release
as well as enabling cross compilation for different arch and OS.

Note that we use musl-gcc as it allows to statically link a libc
implementation while CGO_ENABLED=0 only was leading to the error below:
```
ethermint@v0.22.0-sdk50-1/app/ante/eip712.go:293:36: undefined: secp256k1.RecoverPubkey
ethermint@v0.22.0-sdk50-1/app/ante/eip712.go:319:17: undefined: secp256k1.VerifySignature
```
The statically linked binary is 100M as of today.

Also drop get-diff-action as it's unmaintained and was misbehaving on
the rebasing and squashing workflow, refs Canto-Network#139
@AndreMiras
Copy link
Author

Please merge #139 before so I can rebase on top post merge

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.

1 participant