nveloso commented on code in PR #45470:
URL: https://github.com/apache/arrow/pull/45470#discussion_r1964481777


##########
dev/tasks/python-wheels/github.linux.yml:
##########
@@ -64,15 +64,15 @@ jobs:
         env:
           GITHUB_TOKEN: {{ '${{ secrets.GITHUB_TOKEN }}' }}
         run: |
-          if [ "{{ manylinux_version }}" = "2014" ] && [ "{{ arch }}" = 
"arm64" ]; then
+          if [ "{{ linux_wheel_version }}" = "2014" ] && [ "{{ arch }}" = 
"arm64" ]; then
             # We can't use NuGet on manylinux2014_aarch64 because Mono is old.
             :
-          else
+          elif [ "{{ linux_wheel_name }}" = "many" ]; then

Review Comment:
   I checked the `install_vcpkg.sh` script and it tries to use mono, that's 
probably why you asked me to install mono.
   It's possible to install by building the sources but it takes quite a while. 
The other alternative is to install it via apk but we need to add the testing 
repository and the mono package is not in the main repository. Maybe we can 
install it via apk to improve CI speed. Does this sound right? Do you see a 
problem with adding the testing repository?



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