Thriftpy / thriftpy2

Pure python approach of Apache Thrift.
MIT License
567 stars 90 forks source link

Cython issue still occurring because the fix wasn't published #224

Closed ofiry-tm closed 10 months ago

ofiry-tm commented 11 months ago

I get the Cython issue in the latest published version 0.4.16:

Error compiling Cython file:
------------------------------------------------------------
...

    cdef int grow(self, int min_size):
        if min_size <= self.buf_size:
            return 0

        cdef int multiples = min_size / self.buf_size
                                      ^
------------------------------------------------------------

thriftpy2/transport/cybase.pyx:90:38: Cannot assign type 'double' to 'int'

it seemed to be fixed now on master branch: https://github.com/Thriftpy/thriftpy2/commit/202ed5ef7b91399b86f8b3b76108a14612b38dbc

but it was never published as a new version, can you please publish a new version ?

ofiry-tm commented 11 months ago

@ethe @aisk please help

ethe commented 11 months ago

v0.14.7 has released, thanks

ofiry-tm commented 11 months ago

@ethe I see that you merged the version change, but the latetst release version is still v0.14.6

aisk commented 10 months ago

The latest version is v0.14.7 https://pypi.org/project/thriftpy2/#history .