Closed fselmo closed 1 year ago
@fjl or @karalabe - can we get someone to take a look at this? We need it to be able to support the latest python versions in py-evm
. Thank you!
Thanks @LefterisJP! @fselmo and I just got merge permissions yesterday. I believe he tested it, but will wait for him to confirm before merging. He'll be back on Monday.
Thanks @LefterisJP! @fselmo and I just got merge permissions yesterday. I believe he tested it, but will wait for him to confirm before merging. He'll be back on Monday.
I did test this. We can merge it 👍🏼
Can you push this to pip? Running in this error now, installed it today. Got a version from like 2015.
Can you push this to pip? Running in this error now, installed it today. Got a version from like 2015.
I second this.
What was wrong?
PY_SSIZE_T_CLEAN
defined before includingPython.h
(https://docs.python.org/3/c-api/arg.html#strings-and-buffers)Solution / changes
PY_SSIZE_T_CLEAN
before includingPython.h