kou commented on code in PR #12763:
URL: https://github.com/apache/arrow/pull/12763#discussion_r894142479


##########
dev/tasks/python-wheels/github.osx.arm64.yml:
##########
@@ -134,6 +134,8 @@ jobs:
           $PYTHON -m venv test-arm64-env
           source test-arm64-env/bin/activate
           pip install --upgrade pip wheel
+          arch -arm64 pip install -r arrow/python/requirements-wheel-test.txt

Review Comment:
   Because the `$PYTHON` 
(`/Library/Frameworks/Python.framework/Versions/3.10/bin/python3.10`) is a x86 
Python.
   If we don't use `arch -arm64`, wheels for x86 are installed.
   
   See 
https://github.com/ursacomputing/crossbow/runs/6165284883?check_suite_focus=true#step:10:147
 for example:
   
   ```text
     Using cached MarkupSafe-2.1.1-cp310-cp310-macosx_10_9_x86_64.whl (13 kB)
   ```
   
   We can omit `arch -x86_64` in "Test Wheel on AMD64" but I think that we 
should specify `arch -x86_64` explicitly to work on macOS that provides arm64 
system Python too.



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