use cython for python bindings (#7)

Author: phoenix-o

python/.gitignore

@@ -1,7 +1,9 @@
 *.pyc
 *.so
+pylibra.c
+build/
 /*.egg-info
 /_skbuild
 /.eggs
 /dist
-/venv
+/venv

python/CMakeLists.txt

@@ -0,0 +1,14 @@
+# distutils: include_dirs = ../libra-dev/include/
+
+cdef extern from "data.h":
+    ctypedef struct CDevAccountResource:
+        long balance
+
+    CDevAccountResource account_resource_from_lcs(char* buffer, int length)
+
+
+cdef class DevAccountResource:
+    cdef public long balance
+
+    def __cinit__(self, lcs_bytes):
+        self.balance = account_resource_from_lcs(lcs_bytes, len(lcs_bytes)).balance

python/pylibra.pyx

@@ -1,20 +1,31 @@
 import sys
 
-from skbuild import setup  # This line replaces 'from setuptools import setup'
+from setuptools import setup, Extension
+from Cython.Build import cythonize
 
 # Require pytest-runner only when running tests
-pytest_runner = (['pytest-runner']
-                 if any(arg in sys.argv for arg in ('pytest', 'test'))
-                 else [])
+pytest_runner = (
+    ["pytest-runner"] if any(arg in sys.argv for arg in ("pytest", "test")) else []
+)
 
 setup_requires = pytest_runner
 
-setup(name="pylibra",
-      version="1.0.0",
-      description="Python interface for the libra-dev library function",
-      author="Yucong Sun",
-      author_email="[email protected]",
-      packages=['pylibra'],
-      tests_require=['pytest', 'pytest-runner'],
-      setup_requires=setup_requires
-)
+setup(
+    name="pylibra",
+    version="1.0.0",
+    description="Python interface for the libra-dev library function",
+    author="Yucong Sun",
+    author_email="[email protected]",
+    tests_require=["pytest", "pytest-runner"],
+    ext_modules=cythonize(
+        [
+            Extension(
+                "pylibra",
+                ["pylibra.pyx"],
+                extra_link_args=["-L../libra-dev/target/debug"],
+                libraries=["libra_dev"],
+            )
+        ]
+    ),
+    setup_requires=setup_requires,
+)

python/pylibra/__init__.py

@@ -4,7 +4,7 @@ set -euo pipefail
 python3 -m venv ./venv
 
 ./venv/bin/pip3 install --upgrade pip
-./venv/bin/pip3 install --upgrade scikit-build pytest
+./venv/bin/pip3 install --upgrade pytest cython
 
 ./venv/bin/python3 setup.py develop
 ./venv/bin/python3 setup.py pytest

python/pylibra/_pylibra.c

@@ -1,7 +1,6 @@
-import pylibra
+from pylibra import DevAccountResource
 
 BLOB = bytes.fromhex("020000002100000001674deac5e7fca75f00ca92b1ba3697f5f01ef585011beea7b361150f4504638f0800000002000000000000002100000001a208df134fefed8442b1f01fab59071898f5a1af5164e12c594de55a7004a91c8e0000002000000036ccb9ba8b4f0cd1f3e2d99338806893dff7478c69acee9b8e1247c053783a4800e876481700000000000200000000000000200000000b14ed4f5af8f8f077c7ec4313c6d395b9a7eb5f41eab9ec15367215ca9e420a01000000000000002000000032f56f77b09773aa64c78ee39943da7ec73f91cd757e325098e11b3edc4eccb10100000000000000")
 
 def test_account_state_blob():
-    res = pylibra.account_resource_from_lcs(BLOB)
-    assert res["balance"] == 100000000000
+    assert DevAccountResource(BLOB).balance == 100000000000