terencehonles commented on a change in pull request #8386:
URL: https://github.com/apache/arrow/pull/8386#discussion_r501463131



##########
File path: dev/tasks/python-wheels/travis.osx.yml
##########
@@ -31,7 +31,7 @@ addons:
       - git
       - [email protected]
       - protobuf
-      - [email protected]
+      - [email protected]
       - thrift
       - wget
     update: false

Review comment:
       I think it's possible Travis doesn't have Python 3.9 in it's homebrew 
cache, but I would expect a different error than it succeeds and then 
`/usr/local/opt/[email protected]/libexec/bin/pip` is just not found...
   
   Nevermind! Apparently it's just a "warning" :unamused: :
   > Warning: '[email protected]' formula is unreadable: No available formula with the 
name "[email protected]" 
   
   https://travis-ci.org/github/ursa-labs/crossbow/builds/733873746#L145

##########
File path: python/pyproject.toml
##########
@@ -19,7 +19,9 @@
 requires = [
     "cython >= 0.29",
     "numpy==1.14.5; python_version<'3.7'",
-    "numpy==1.16.0; python_version>='3.7'",

Review comment:
       Just to follow up, 1.19.2 did work for me, I'm just saying there might 
be a earlier release that works. It's probably simplest/best to use 1.19.2 
since it's mentioned in their changelog:
   
   > This release supports Python 3.6-3.8. Cython >= 0.29.21 needs to be used 
when building with Python 3.9 for testing purposes.
   https://numpy.org/devdocs/release/1.19.2-notes.html
   
   ...looking at 1.19.1 it says the same thing so it might be safe to go 
backwards




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


Reply via email to