AlenkaF commented on code in PR #47313:
URL: https://github.com/apache/arrow/pull/47313#discussion_r2281725694


##########
ci/docker/linux-apt-python-313-freethreading.dockerfile:
##########
@@ -32,13 +32,18 @@ COPY python/requirements-build.txt \
 
 ENV ARROW_PYTHON_VENV /arrow-dev
 RUN python3.13t -m venv ${ARROW_PYTHON_VENV}
+
+# Free-threaded CPython 3.13 needs pinned cffi version
+RUN sed '/^cffi/d' /arrow/python/requirements-test.txt > 
/arrow/python/requirements-patched.txt && \
+    echo "cffi<2.0.0" >> /arrow/python/requirements-patched.txt

Review Comment:
   It is surprising, yeah. In the examples shared patch versions are not used 
in the `python_version` Environment Marker. For example, [this 
issue](https://github.com/python/peps/issues/560) led me to believe that 
`python_version` only considers the major and minor versions.
   
   As for the `t` suffix, I’m also not entirely sure about the background 
there. I vaguely remember hearing at EuroPython that it's going to stay, but I 
could be misremembering 😬



-- 
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-unsubscr...@arrow.apache.org

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

Reply via email to