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

[Python] Cython compilation error on cython==0.27.3 #20626

Closed
asfimport opened this issue Dec 14, 2018 · 4 comments
Closed

[Python] Cython compilation error on cython==0.27.3 #20626

asfimport opened this issue Dec 14, 2018 · 4 comments

Comments

@asfimport
Copy link

On the latest master, I'm getting the following error:

[ 11%] Compiling Cython CXX source for lib...



Error compiling Cython file:

------------------------------------------------------------

...



    out.init(type)

    return out





cdef object pyarrow_wrap_metadata(

    ^

------------------------------------------------------------



pyarrow/public-api.pxi:95:5: Function signature does not match previous declaration

CMakeFiles/lib_pyx.dir/build.make:57: recipe for target 'CMakeFiles/lib_pyx' failed

With 0.29.0 it is working. This might have been introduced in 1220184 but I'm not sure.

Reporter: Philipp Moritz / @pcmoritz
Assignee: Uwe Korn / @xhochy

PRs and other links:

Note: This issue was originally created as ARROW-4024. Please see the migration documentation for further details.

@asfimport
Copy link
Author

Uwe Korn / @xhochy:
@pcmoritz Is there anything that keeps you on this Cython version? As Cython is a build-time but not a runtime dependency, I would close this bug as won't fix (maybe first ensure that we pin correctly).

@asfimport
Copy link
Author

Philipp Moritz / @pcmoritz:
I think it's fine to drop support for 0.27. I just wanted to point it out because from

if Cython.__version__ < '0.27':

setup_requires=['setuptools_scm', 'cython >= 0.27'] + setup_requires,

it looks like 0.27 is supported.

@asfimport
Copy link
Author

Wes McKinney / @wesm:
Let's bump the minimum Cython version

@asfimport
Copy link
Author

Wes McKinney / @wesm:
Issue resolved by pull request 3590
#3590

@asfimport asfimport added this to the 0.13.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants