Skip to content
This repository has been archived by the owner on May 13, 2022. It is now read-only.

Hacked away until buildable and runnable with Tendermint 0.8.0 #502

Conversation

silasdavis
Copy link
Contributor

This gets feature-tendermint_0.8.0 to a state where:

  • Tendermint and its transitive dependencies are upgraded 0.8.0
  • All reference to tmsp have been converted to abci references
  • Our partial support for out-of-process tmsp has been removed (we can add back in the abci server stuff if we want)
  • EventSwitch has been changed to an interface throughout
  • A Tendermint node can start but fails on making the cs_wal_dir

feature-tendermint_0.8.0 is a WIP branch with failing tests. So this is potentially mergeable despite failing tests. We may be able to find a way to divide work at this point, or it may be easier for me to try and get a bit further.

@silasdavis silasdavis changed the title Hacked away until buildable and runnable with 0.8.0 Hacked away until buildable and runnable with Tendermint 0.8.0 Feb 21, 2017
@benjaminbollen
Copy link
Contributor

I will merge release-0.16 into eris-ltd:feature-tendermint_0.8.0; so that this PR gives a clearer picture of what has been done and where we are

@silasdavis
Copy link
Contributor Author

have rebased against eris-ltd/feature-tendermint_0.8.0

@@ -55,7 +55,7 @@ func Multiplex(events ...EventEmitter) *multiplexedEvents {

// The events struct has methods for working with events.
type events struct {
eventSwitch *go_events.EventSwitch
eventSwitch go_events.EventSwitch
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is this changed; a requirement or an improvement?

@@ -1,4 +1,3 @@
// +build integration

// Space above here matters
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

best to remember to remove //space above here matters now ;)

# listener address for accepting tendermint socket protocol connections
listener = "tcp://0.0.0.0:46658"

################################################################################
##
##yeah we had partial support for that with TMSP
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can remove this from the config

@benjaminbollen benjaminbollen merged commit 5000562 into hyperledger-archives:feature-tendermint_0.8.0 Feb 23, 2017
silasdavis pushed a commit to silasdavis/burrow that referenced this pull request Mar 9, 2019
…endermint_0.8.0

Hacked away until buildable and runnable with Tendermint 0.8.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants