Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix check_supplements assert error #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

YadominJinta
Copy link

shelve.open may create db with extension, on Windows, it creates abi_db.shelve.dat, and abi_path().is_file() will always be False.

Refer: https://docs.python.org/3/library/shelve.html#shelve.open
Fix #61

@srbaeza-ai
Copy link

#72

@EasonC13
Copy link

Thank you for fixing this! Hope this can be merged soon!

@srbaeza-ai
Copy link

👍

@EasonC13
Copy link

@YadominJinta Thank you for your work.
I tried to install your package and run it, but it stuck at Running light execution to find functions.
Can you check it out?

> WEB3_PROVIDER_URI=https://eth.llamarpc.com panoramix 0x49091eeaf9A713daBcD0B96D4C12749B218A52a0

2024-02-22 05:50:44,530 HOST panoramix.loader[2685642] INFO Fetching code for 0x49091eeaf9a713dabcd0b96d4c12749b218a52a0...
2024-02-22 05:50:46,944 HOST panoramix.decompiler[2685642] INFO Running light execution to find functions.

@srbaeza-ai
Copy link

#72

@srbaeza-ai
Copy link

I'm going to work on it I'll get back to you

@YadominJinta
Copy link
Author

@EasonC13 if you're using powershell, try

$env:WEB3_PROVIDER_URI="https://eth.llamarpc.com"
panoramix 0x49091eeaf9A713daBcD0B96D4C12749B218A52a0

I just fixed the shelve db problem, but this decompiler still have a lot of compatible problems with Windows, so I suggest using WSL if you need it.

Copy link

@Dioliode Dioliode left a comment

Choose a reason for hiding this comment

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

DioX

Copy link

@Dioliode Dioliode left a comment

Choose a reason for hiding this comment

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

OdiX

Copy link

@Dioliode Dioliode left a comment

Choose a reason for hiding this comment

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

DioX

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AssertionError
5 participants