Skip to content
This repository has been archived by the owner on Oct 4, 2019. It is now read-only.

1.4.11 stable #37

Merged
merged 2 commits into from
Sep 20, 2016
Merged

1.4.11 stable #37

merged 2 commits into from
Sep 20, 2016

Conversation

elaineo
Copy link
Member

@elaineo elaineo commented Sep 20, 2016

Trying to get a clean build to work from.

The only change since 1.4.10 is the ETH hotfix to fix the memory leak, with added comments to the best of my ability

Also removed the dao_test so all tests pass.

@elaineo
Copy link
Member Author

elaineo commented Sep 20, 2016

I gave up on trying to get all the tests working on the most up-to-date version, so I applied the memory-leak patch to the last stable release. If @whatisgravity returns, or if someone else wants to take a stab at it, we can merge all the subsequent changes back in, but right now we should have a build release that passes all the tests.

Copy link
Contributor

@splix splix left a comment

Choose a reason for hiding this comment

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

I can only confirm that this patch is equal to the corresponding fix in forked ethereum (commit ethereum/go-ethereum@e44b2dc)

As it didn't make any problems to ETH users, it's safe to merge in our codebase

@elaineo
Copy link
Member Author

elaineo commented Sep 20, 2016

I'd like to send this straight to a release. I don't see the point of merging it into develop since develop is currently unstable.

@splix
Copy link
Contributor

splix commented Sep 20, 2016

@elaineo ok, that's probably a good idea

@mikeyb
Copy link
Contributor

mikeyb commented Sep 20, 2016

I don't see an issue merging to develop. This change doesnt affect anything currently out of sync in develop. We can cherry pick the commit like arachnid suggested to master and promote into a release that way while still retaining the fix in develop in case we decide to stick with develop branch

@splix splix merged commit 22df8a2 into develop Sep 20, 2016
@elaineo elaineo deleted the 1.4.11-stable branch September 28, 2016 03:18
soc1c added a commit that referenced this pull request Aug 13, 2019
* Replace repository reference for accounts/abi/bind (#29)

* docs: update badges (#34)

* docs: add gitter badge

* docs: add circle-ci badge

* Update README.md

* Update README.md

* Update README.md

* docs: update repository path

* Update README.md

* Tx Encoding and decoding fix (#33)

* Added test for decoding old encoding function

* Fixed decoding issue and improved tests to handle all cases including previous client

* Add compatibility with previous version

* Rename receipt storage variable

* Storage and edge case fixes (#28)

* Applied storage fixes and updates

* Minor execution refactor
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants