[Bug]: Incorrect example to set multiple denoms for mininum-gas-prices

[pwnfoo]

Is there an existing issue for this?

• I have searched the existing issues

What happened?

What?

The example comment in app.toml uses ; as the separator for specifying multiple denominations for minimum gas price.

Following the example leads to a panic when the application starts :

panic: invalid minimum gas prices: invalid decimal coin expression: 0.025ufoo;0.025ubar

This issue arises because the SetMinGasPrices function calls ParseDecCoins, which uses , as the separator

Solution

• Update the example comment to

# transaction. A transaction's fees must meet the minimum of any denomination
# specified in this config (e.g. 0.25token1,0.0001token2).

• Modify GetMinGasPrices to use ',' as the separator

[julienrbrt]

I think the issues comes from

cosmos-sdk/server/util.go

Line 524 in e8da7ac

baseapp.SetMinGasPrices(cast.ToString(appOpts.Get(FlagMinGasPrices))),

.

The config getter and setter seem to be working both properly with ; delimitation.
That does not change the bug, and they should both behave the same for sure.