Skip to content

Not tested on PyPy. With current state of ctypes in PyPy it is unlikely to be worth to support. #38

@dgpv

Description

@dgpv

pypy3 -u -m unittest bitcointx.tests.test_scripteval.Test_EvalScript.test_script_bitcoinconsensus

fails with _ctypes.basics.ArgumentError: Expected bytes of length one as character

while the arguments are clearly multi-byte (but even if it was bytes of length 1, this error should not be raised, in my understanding).

It seems that ctypes is a second-class citizen in PyPy land, and cffi is preferred, while python-bitcointx uses ctypes and there's no point in using more heavy ffi package.

Debugging these quirks is too costly.

Because of this, I removed PyPy from the tests in .travis.yml in 302f4f3, to indicate that python-bitcointx cannot be expected to work correctly with PyPy.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions