terencehonles commented on a change in pull request #8916:
URL: https://github.com/apache/arrow/pull/8916#discussion_r554301286
##########
File path: dev/tasks/python-wheels/win-build.bat
##########
@@ -22,11 +22,19 @@
@rem --file=arrow\ci\conda_env_gandiva.yml ^
-@rem create conda environment for compiling
@rem FIXME: Update to numpy 1.16.6
+SET NUMPY_VERSION=1.16.5
+SET VS_VERSION=
+if %PYTHON_VERSION% == 3.6 (
+ SET VS_VERSION="<14.16"
Review comment:
I'm glad I don't need to deal with batch scripting on a regular basis,
but it seems like this should have been `SET VS_VERSION=^<14.16` based on
https://docs.microsoft.com/en-us/windows-server/administration/windows-commands/set_1#remarks
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
