WIP fix gen-tx

[rigelrozanski]

This PR should close #1804

breaking changes were made in #1373 which broke the gen-tx mechanism in init.

Example commands which demonstrate the break:

gaiad init gen-tx --name=foo --home=$HOME/foobar                                           
 .............. > respond with some JSON indicating success 
gaiad init --gen-txs -o --home=$HOME/foobar
ERROR: Error{The genesis file must have at least one validator}

That final error should not be occurring. (basically non of the validators from any gen-tx are being processed and added to the genesis.json)

The current roll-back (aka my first commit here) has the Tendermint validators in the genesis file, which I think we've already removed? Not totally sure, I'm assuming that still needs to be updated in this PR @mossid

• Linked to github-issue with discussion and accepted design
• Updated all relevant documentation (docs/)
• Updated all relevant code comments
• Wrote tests
• Added entries in PENDING.md
• Updated cmd/gaia and examples/

---

For Admin Use:

• Added appropriate labels to PR (ex. wip, ready-for-review, docs)
• Reviewers Assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[codecov]

Codecov Report

Merging #1824 into develop will decrease coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop    #1824      +/-   ##
===========================================
- Coverage    63.44%   63.42%   -0.02%     
===========================================
  Files          117      117              
  Lines         6938     6940       +2     
===========================================
  Hits          4402     4402              
- Misses        2281     2283       +2     
  Partials       255      255

[alexanderbez]

This also breaks the make localnet-stop/start I believe.

@cwgoes does this relate to what I brought up earlier? Something along the lines of ResponseInitChain was brought up?

[cwgoes]

Huh I don't think we ought to need this - #1373, which is merged, should mean that the validators are set on ResponseInitChain according to their power.

Have we tried just removing the assertion (or is it Tendermint-side)?

edit: Aha, see tendermint/tendermint#2015.

[cwgoes]

Tested ACK

Ref #1840 once upstream is updated.