Skip to content
This repository has been archived by the owner on Jul 1, 2021. It is now read-only.

Upgrade py-evm to v0.3.0a9 #1343

Merged
merged 3 commits into from Dec 3, 2019
Merged

Upgrade py-evm to v0.3.0a9 #1343

merged 3 commits into from Dec 3, 2019

Conversation

carver
Copy link
Contributor

@carver carver commented Dec 2, 2019

What was wrong?

A bug in the Istanbul implementation is in the pinned version of py-evm.

How was it fixed?

Upgrade py-evm to latest version which fixes the bug:
https://py-evm.readthedocs.io/en/latest/release_notes.html#py-evm-0-3-0-alpha-9-2019-12-02

To-Do

  • Clean up commit history

Cute Animal Picture

put a cute animal picture link inside the parentheses

@carver
Copy link
Contributor Author

carver commented Dec 2, 2019

The wheel failures are real, from a recent ipython release. We were hooking into some non-public API which disappeared, and prevented us from attaching. So all the trinity attach commands were failing.

@carver
Copy link
Contributor Author

carver commented Dec 2, 2019

Ok, so ipython 7.10.1 gives a mysterious blinking Press ENTER to continue.... 7.10.0 shows what's going on by spitting out this warning over and over:

Press ENTER to continue...                                                                                                                                                                                                                    
Exception 'Loop' object has no attribute '_ready'

Unhandled exception in event loop:
  File "uvloop/cbhandles.pyx", line 68, in uvloop.loop.Handle._run
  File "/tmp/tmphvl_lsv2/package-smoke-test/lib/python3.6/site-packages/prompt_toolkit/eventloop/utils.py", line 69, in schedule
    if not loop2._ready:  # type: ignore
Press ENTER to continue...                                                                                                                                                                                                                    

Exception 'Loop' object has no attribute '_ready'

...(etc)

So it looks like it's our prompt-toolkits fault for hooking into some private API, which is allowed to change in a non-major release. I'll pin to <7.10 for now.

ipython v7.10.1 gives a mysterious blinking `Press ENTER to continue...`.
v7.10.0 shows what's going on by spitting out this exception over and over:

Press ENTER to continue...
Exception 'Loop' object has no attribute '_ready'

Unhandled exception in event loop:
  File "uvloop/cbhandles.pyx", line 68, in uvloop.loop.Handle._run
  File "/tmp/fresh-venv/lib/python3.6/site-packages/prompt_toolkit/eventloop/utils.py", line 69, in schedule
    if not loop2._ready:  # type: ignore
Press ENTER to continue...

...(etc)

I haven't had a chance to hunt down if we have another fix for it, so
for now I'm treating it like an ipython bug. Once attach works,
we can relax ipython versions again.

More discussion here:
ethereum#1343
Copy link
Contributor

@cburgdorf cburgdorf left a comment

Choose a reason for hiding this comment

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

:shipit:

@carver carver merged commit 2765a61 into ethereum:master Dec 3, 2019
@carver carver deleted the upgrade-pyevm branch December 3, 2019 18:33
@carver
Copy link
Contributor Author

carver commented Dec 4, 2019

Closed #1312

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.

None yet

2 participants