raulcd commented on code in PR #48618:
URL: https://github.com/apache/arrow/pull/48618#discussion_r2727444219


##########
ci/scripts/python_wheel_macos_build.sh:
##########
@@ -177,6 +177,11 @@ export CMAKE_PREFIX_PATH=${build_dir}/install
 export SETUPTOOLS_SCM_PRETEND_VERSION=${PYARROW_VERSION}
 
 pushd ${source_dir}/python
+# We first populate stub docstrings and then build the wheel
+python setup.py build_ext --inplace

Review Comment:
   Won't this build pyarrow twice one?
   `python setup.py build_ext --inplace`
   and one for
   `python setup.py bdist_wheel`
   
   I'll take a look at that but, if that's the case, we probably want to check 
whether there are alternatives to not build twice.
   
   I am running wheels at the moment to validate them.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to