You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "joosthooz (via GitHub)" <gi...@apache.org> on 2023/04/06 14:21:47 UTC

[GitHub] [arrow] joosthooz opened a new issue, #34933: Building pyarrow does not work with all Cython 0.29.* versions

joosthooz opened a new issue, #34933:
URL: https://github.com/apache/arrow/issues/34933

   ### Describe the bug, including details regarding any error messages, version, and platform.
   
   Building pyarrow results in errors when I use Cython 0.29.24, but succeeds when using the latest 0.29.34.
   In `requirements-build.txt` the version is `cython>=0.29` so maybe that should be updated to a later version?
   
   ### Component(s)
   
   Python


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] jorisvandenbossche commented on issue #34933: [Python] Building pyarrow does not work with all Cython 0.29.* versions

Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #34933:
URL: https://github.com/apache/arrow/issues/34933#issuecomment-1499250404

   Testing a few versions, 0.29.31 seems to be the first one that doesn't have the above mentioned cythonization error. So we can maybe use that one as minimum version


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] jorisvandenbossche commented on issue #34933: [Python] Building pyarrow does not work with all Cython 0.29.* versions

Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #34933:
URL: https://github.com/apache/arrow/issues/34933#issuecomment-1499240238

   Yes, we can probably be more exact in the minimum version requirement for cython in our build requirements (for example, I know pandas regularly updates that and has "Cython>=0.29.33" in their pyproject.toml build-system.requires.
   
   We also have a check in setup.py to enforce a minimum version, which we should update as well (and keep in sync):
   
   https://github.com/apache/arrow/blob/84e54308f1399df094b785083e6faf52275d10e9/python/setup.py#L43-L44
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] jorisvandenbossche closed issue #34933: [Python] Building pyarrow does not work with all Cython 0.29.* versions

Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche closed issue #34933: [Python] Building pyarrow does not work with all Cython 0.29.* versions
URL: https://github.com/apache/arrow/issues/34933


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] joosthooz commented on issue #34933: Building pyarrow does not work with all Cython 0.29.* versions

Posted by "joosthooz (via GitHub)" <gi...@apache.org>.
joosthooz commented on issue #34933:
URL: https://github.com/apache/arrow/issues/34933#issuecomment-1499147630

   Example of the errors: 
   ```
           -- Running cmake --build for PyArrow
           cmake --build . --config release -- -j4
           [  1%] Compiling Cython CXX source for lib...
           [  2%] Compiling Cython CXX source for _compute...
           [  4%] Compiling Cython CXX source for _feather...
           [  5%] Compiling Cython CXX source for _csv...
           
           Error compiling Cython file:
           ------------------------------------------------------------
           ...
           
           cdef extern from "Python.h":
               int PySlice_Check(object)
           
           
           cdef int check_status(const CStatus& status) except -1 nogil
                                                                 ^
           ------------------------------------------------------------
           
           pyarrow/lib.pxd:67:55: Syntax error in C variable declaration
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org