WillAyd commented on code in PR #45854:
URL: https://github.com/apache/arrow/pull/45854#discussion_r2018833185


##########
.github/workflows/dev.yml:
##########
@@ -124,7 +124,7 @@ jobs:
         shell: bash
         run: |
           gem install test-unit
-          pip install "cython>=3" setuptools pytest requests setuptools-scm
+          pip install "cython>=3" meson-python "cmake<4" pytest requests 
meson-python setuptools-scm

Review Comment:
   It looks like cmake=4.0.0 was just released today and is causing some issues 
with mimalloc and boost resolution. Hope that this upper bound won't have to 
stay long



##########
python/requirements-test.txt:
##########
@@ -4,3 +4,4 @@ pandas
 pytest
 pytz
 pyuwsgi; sys.platform != 'win32' and python_version < '3.13'
+setuptools>=64

Review Comment:
   While setuptools isn't required to build anymore, there are a few tests that 
create inline setup.py files and use them to test the building of third party 
libraries. It might be nice to port those off of setuptools, but I don't think 
that is critical for the initial PR



-- 
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