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