You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/07/16 11:52:16 UTC

[GitHub] [arrow] kszucs opened a new pull request #7785: [Packaging][Python] Fix macOS wheel build failures

kszucs opened a new pull request #7785:
URL: https://github.com/apache/arrow/pull/7785


   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on pull request #7785: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#issuecomment-659361027


   @github-actions crossbow submit wheel-osx-cp37m wheel-osx-cp35m


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on a change in pull request #7785: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on a change in pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#discussion_r455742672



##########
File path: dev/tasks/python-wheels/travis.osx.yml
##########
@@ -74,8 +72,8 @@ install:
 
   # crossbow dependencies for deployment
   - unset MACOSX_DEPLOYMENT_TARGET
-  - pip3 install click ruamel.yaml setuptools_scm github3.py toolz requests[security]
-  - python3 arrow/dev/tasks/crossbow.py
+  - /usr/local/bin/pip3 install click ruamel.yaml setuptools_scm github3.py toolz requests[security]
+  - /usr/local/bin/python3 arrow/dev/tasks/crossbow.py

Review comment:
       There were ssl issues during the artifact uploading in the 3.5 build. I hope that using the brew python will solve that.




----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs closed pull request #7785: ARROW-9506: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs closed pull request #7785:
URL: https://github.com/apache/arrow/pull/7785


   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] github-actions[bot] commented on pull request #7785: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#issuecomment-659364085


   Revision: 60ab09e1bc0808f0b46b8f45cd4619871d30ec7f
   
   Submitted crossbow builds: [ursa-labs/crossbow @ actions-426](https://github.com/ursa-labs/crossbow/branches/all?query=actions-426)
   
   |Task|Status|
   |----|------|
   |wheel-osx-cp35m|[![TravisCI](https://img.shields.io/travis/ursa-labs/crossbow/actions-426-travis-wheel-osx-cp35m.svg)](https://travis-ci.org/ursa-labs/crossbow/branches)|
   |wheel-osx-cp37m|[![TravisCI](https://img.shields.io/travis/ursa-labs/crossbow/actions-426-travis-wheel-osx-cp37m.svg)](https://travis-ci.org/ursa-labs/crossbow/branches)|


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on a change in pull request #7785: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on a change in pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#discussion_r455742672



##########
File path: dev/tasks/python-wheels/travis.osx.yml
##########
@@ -74,8 +72,8 @@ install:
 
   # crossbow dependencies for deployment
   - unset MACOSX_DEPLOYMENT_TARGET
-  - pip3 install click ruamel.yaml setuptools_scm github3.py toolz requests[security]
-  - python3 arrow/dev/tasks/crossbow.py
+  - /usr/local/bin/pip3 install click ruamel.yaml setuptools_scm github3.py toolz requests[security]
+  - /usr/local/bin/python3 arrow/dev/tasks/crossbow.py

Review comment:
       There were ssl issues in the 3.5 build. I hope using the brew python will solve that.




----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on pull request #7785: ARROW-9506: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#issuecomment-659606947


   Luckily the build times have improved by ~8 minutes.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on a change in pull request #7785: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on a change in pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#discussion_r455742169



##########
File path: dev/tasks/python-wheels/travis.osx.yml
##########
@@ -64,7 +62,7 @@ install:
   # install the built wheel and test dependencies
   - install_wheel arrow
   # run unit tests before removing the system libraries
-  - run_unit_tests arrow
+  - ulimit -S -n 4096 && run_unit_tests arrow

Review comment:
       Python 3.7 tests were failing because of too many open files.




----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on a change in pull request #7785: ARROW-9506: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on a change in pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#discussion_r456007128



##########
File path: dev/tasks/python-wheels/travis.osx.yml
##########
@@ -18,6 +18,35 @@ os: osx
 osx_image: xcode11.3
 language: generic
 
+addons:
+  homebrew:
+    packages:
+      - automake

Review comment:
       We can reduce the preinstalled dependencies because (for some of them) we use bundled alternatives, but I wouldn't like to do more roundtrips with travis before cutting RC1.




----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] github-actions[bot] commented on pull request #7785: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#issuecomment-659465395


   Revision: 981632a97bacf594277ae29339ccae8d8fb21e0f
   
   Submitted crossbow builds: [ursa-labs/crossbow @ actions-427](https://github.com/ursa-labs/crossbow/branches/all?query=actions-427)
   
   |Task|Status|
   |----|------|
   |wheel-osx-cp35m|[![TravisCI](https://img.shields.io/travis/ursa-labs/crossbow/actions-427-travis-wheel-osx-cp35m.svg)](https://travis-ci.org/ursa-labs/crossbow/branches)|
   |wheel-osx-cp37m|[![TravisCI](https://img.shields.io/travis/ursa-labs/crossbow/actions-427-travis-wheel-osx-cp37m.svg)](https://travis-ci.org/ursa-labs/crossbow/branches)|


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on pull request #7785: ARROW-9506: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#issuecomment-659605453


   Ran the builds manually:
   - [cp37m](https://travis-ci.org/github/ursa-labs/crossbow/builds/708848647): green
   - [cp35m](https://travis-ci.org/github/ursa-labs/crossbow/builds/708848617): green
   - [cp36m](https://travis-ci.org/github/ursa-labs/crossbow/builds/708867586): pending
   - [cp38](https://travis-ci.org/github/ursa-labs/crossbow/builds/708869667): pending


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on pull request #7785: ARROW-9506: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#issuecomment-659619537


   All green, merging.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on pull request #7785: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#issuecomment-659463129


   @github-actions crossbow submit wheel-osx-cp37m wheel-osx-cp35m


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] kszucs commented on a change in pull request #7785: ARROW-9506: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
kszucs commented on a change in pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#discussion_r456007128



##########
File path: dev/tasks/python-wheels/travis.osx.yml
##########
@@ -18,6 +18,35 @@ os: osx
 osx_image: xcode11.3
 language: generic
 
+addons:
+  homebrew:
+    packages:
+      - automake

Review comment:
       We can reduce the preinstalled dependencies because for some of them we bundled alternative, but I wouldn't like to do more roundtrips with travis before cutting RC1.




----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [arrow] github-actions[bot] commented on pull request #7785: ARROW-9506: [Packaging][Python] Fix macOS wheel build failures

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #7785:
URL: https://github.com/apache/arrow/pull/7785#issuecomment-659610053


   https://issues.apache.org/jira/browse/ARROW-9506


----------------------------------------------------------------
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:
users@infra.apache.org