From 9e1d28e2a60b51df9f0c6b91d8c6c553339c3f6a Mon Sep 17 00:00:00 2001 From: Mark Rushakoff Date: Tue, 11 Jun 2024 04:31:06 -0400 Subject: [PATCH] chore: write gentx info to cmd.ErrOrStderr (#20616) --- CHANGELOG.md | 1 + x/genutil/client/cli/collect.go | 2 +- x/genutil/client/cli/init.go | 7 ++++--- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 21784defaa..e5d1ce7be6 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -186,6 +186,7 @@ Every module contains its own CHANGELOG.md. Please refer to the module you are i * (server) [#20140](https://github.com/cosmos/cosmos-sdk/pull/20140) Remove embedded grpc-web proxy in favor of standalone grpc-web proxy. [Envoy Proxy](https://www.envoyproxy.io/docs/envoy/latest/start/start) * (client) [#20255](https://github.com/cosmos/cosmos-sdk/pull/20255) Use comet proofOp proto type instead of sdk version to avoid needing to translate to later be proven in the merkle proof runtime. * (all) [#19726](https://github.com/cosmos/cosmos-sdk/pull/19726) Integrate comet v1 +* (client) [#20616](https://github.com/cosmos/cosmos-sdk/pull/20616) gentx subcommand output goes to `cmd.ErrOrStderr()` instead of being hardcoded to `os.Stderr` ### Client Breaking Changes diff --git a/x/genutil/client/cli/collect.go b/x/genutil/client/cli/collect.go index 3b0354c107..f8e59533f1 100644 --- a/x/genutil/client/cli/collect.go +++ b/x/genutil/client/cli/collect.go @@ -52,7 +52,7 @@ func CollectGenTxsCmd(genBalIterator types.GenesisBalancesIterator, validator ty toPrint.AppMessage = appMessage - return displayInfo(toPrint) + return displayInfo(cmd.ErrOrStderr(), toPrint) }, } diff --git a/x/genutil/client/cli/init.go b/x/genutil/client/cli/init.go index 920dfcec26..db9491df59 100644 --- a/x/genutil/client/cli/init.go +++ b/x/genutil/client/cli/init.go @@ -5,6 +5,7 @@ import ( "encoding/json" "errors" "fmt" + "io" "os" "path/filepath" @@ -58,13 +59,13 @@ func newPrintInfo(moniker, chainID, nodeID, genTxsDir string, appMessage json.Ra } } -func displayInfo(info printInfo) error { +func displayInfo(dst io.Writer, info printInfo) error { out, err := json.MarshalIndent(info, "", " ") if err != nil { return err } - _, err = fmt.Fprintf(os.Stderr, "%s\n", out) + _, err = fmt.Fprintf(dst, "%s\n", out) return err } @@ -175,7 +176,7 @@ func InitCmd(mm *module.Manager) *cobra.Command { toPrint := newPrintInfo(config.Moniker, chainID, nodeID, "", appState) cfg.WriteConfigFile(filepath.Join(config.RootDir, "config", "config.toml"), config) - return displayInfo(toPrint) + return displayInfo(cmd.ErrOrStderr(), toPrint) }, }