refactor: migrate to cosmos/gogoproto#9356
refactor: migrate to cosmos/gogoproto#9356thanethomson merged 5 commits intotendermint:mainfrom julienrbrt:gogoproto
cosmos/gogoproto#9356Conversation
cosmos/gogoprotocosmos/gogoproto
thanethomson
left a comment
There was a problem hiding this comment.
Who's going to be committing to maintaining this fork over time?
Also, what differences are there between the fork and https://github.com/gogo/protobuf?
I believe that the SDK team will be maintaining it. It is the case that gogo/protobuf is not presently maintained, so it's probably best that we move to something. The tendermint core team has long been of the opinion that Tendermint should move to using the upstream/default protobuf generation rather than any fork, but all other things being equal, the cosmos protobuf generator is probably better than gogo proto, though I'd still prefer to use stock prototobuf. |
|
Cool, thanks for this @julienrbrt 👍 I see some minor conflicts have come up here, but otherwise this should be good to merge. |
Right, the fork adds as well castrepeated support (gogo/protobuf#658). |
Co-authored-by: Thane Thomson <connect@thanethomson.com>
|
if this is merged in a short period could we include it in 0.37? |
thanethomson
left a comment
There was a problem hiding this comment.
Seems reasonable to me. Thanks @julienrbrt!
* refactor: migrate to `cosmos/gogoproto` * add changelog * Update Makefile Co-authored-by: Thane Thomson <connect@thanethomson.com> * update link Co-authored-by: Thane Thomson <connect@thanethomson.com> (cherry picked from commit 101bf50) # Conflicts: # go.mod
|
thank you!! |
* refactor: migrate to `cosmos/gogoproto` * add changelog * Update Makefile Co-authored-by: Thane Thomson <connect@thanethomson.com> * update link Co-authored-by: Thane Thomson <connect@thanethomson.com>
* refactor: migrate to `cosmos/gogoproto` (#9356) * refactor: migrate to `cosmos/gogoproto` * add changelog * Update Makefile Co-authored-by: Thane Thomson <connect@thanethomson.com> * update link Co-authored-by: Thane Thomson <connect@thanethomson.com> * `go mod tidy` Co-authored-by: Thane Thomson <connect@thanethomson.com>
…x-rc1 * release/v0.37.0-rc1: QA Process report for v0.37.x (and baseline for v0.34.x) (tendermint#9499) (tendermint#9577) Fix TX payload for DO testnets (tendermint#9540) (tendermint#9542) blocksync: retry requests after timeout (backport tendermint#9518) (tendermint#9533) Extend the load report tool to include transactions' hashes (tendermint#9509) (tendermint#9513) build(deps): Bump styfle/cancel-workflow-action from 0.10.0 to 0.10.1 (tendermint#9502) build(deps): Bump actions/stale from 5 to 6 (tendermint#9494) loadtime: add block time to the data point (tendermint#9484) (tendermint#9489) config: Add missing storage section when generating config (tendermint#9483) (tendermint#9487) Sync Vote.Verify() in spec with implementation (tendermint#9466) (tendermint#9476) fix spec (tendermint#9467) (tendermint#9469) metrics: fix panic because of absent prometheus label (tendermint#9455) (tendermint#9474) Ensure Dockerfile stages use consistent Go version (backport tendermint#9462) (tendermint#9472) build(deps): Bump slackapi/slack-github-action from 1.21.0 to 1.22.0 (tendermint#9432) build(deps): Bump bufbuild/buf-setup-action from 1.7.0 to 1.8.0 (tendermint#9453) state: restore previous error message (tendermint#9435) (tendermint#9440) build(deps): Bump gonum.org/v1/gonum from 0.11.0 to 0.12.0 (tendermint#9411) docs: Update ADRs for v0.37 (tendermint#9399) (tendermint#9418) build(deps): Bump github.com/spf13/viper from 1.12.0 to 1.13.0 (tendermint#9410) build(deps): Bump github.com/lib/pq from 1.10.6 to 1.10.7 (tendermint#9409) feat: support HTTPS inside websocket (tendermint#9416) (tendermint#9422) Removed unused param (tendermint#9394) test: generate uuid on startup for load tool (tendermint#9383) (tendermint#9392) add redirect links (tendermint#9385) (tendermint#9389) refactor: migrate to cosmos/gogoproto (backport tendermint#9356) (tendermint#9381) cmd: print all versions of tendermint and its sub protocols (tendermint#9329) (tendermint#9386) Add missing changes changelog files (backport tendermint#9376) (tendermint#9382) add separated runs by UUID (backport tendermint#9367) (tendermint#9379) spec: abci++ cleanup for v0.37 (backport tendermint#9288) (tendermint#9374) ci: Remove "(WARNING: BETA SOFTWARE)" tagline from all upcoming releases (tendermint#9371) (tendermint#9372) Update rpc client header (tendermint#9276) (tendermint#9349) ci: Pre-release workflows (backport tendermint#9366) (tendermint#9368) test: add the loadtime report tool (tendermint#9351) (tendermint#9364) Update Tendermint version to v0.37.0 (tendermint#9354) test: add the loadtime tool (tendermint#9342) (tendermint#9357) # Conflicts: # version/version.go
The SDK is moving away from
gogo/protobufto use our forkcosmos/gogoproto. This fork is based on Regen's fork, which we have been using for sometimes (https://github.com/cosmos/cosmos-sdk/blob/main/go.mod#L301).Currently, we are blocked because we are importing TM types (cosmos/cosmos-sdk#13070 (comment)) and I think it would be a win for both to switch to
cosmos/gogoprotoasgogo/protobufis currently not maintained anymore.PR checklist
CHANGELOG_PENDING.mdupdated, or no changelog entry neededdocs/) and code comments, or nodocumentation updates needed