Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore the custom build script for pyarrow #84

Merged
merged 1 commit into from
Aug 5, 2022

Conversation

changhiskhan
Copy link
Contributor

For local development it's much faster to be able do python setup.py develop
So we use the previous thirdparty build script to build pyarrow against
system libarrow that we installed to enable liblance build.

The main change in this PR is changing the arrow ref being checked out. By
default this builds pyarrow 8.0.0.devXXX, which fails the pylance requirement
of pyarrow>=8,<9. Instead we use the 8.0.1 tag in pyarrow

For local development it's much faster to be able do python setup.py develop
So we use the previous thirdparty build script to build pyarrow against
system libarrow that we installed to enable liblance build.

The main change in this PR is changing the arrow ref being checked out. By
default this builds pyarrow 8.0.0.devXXX, which fails the pylance requirement
of pyarrow>=8,<9. Instead we use the 8.0.1 tag in pyarrow
@changhiskhan changhiskhan requested a review from eddyxu August 5, 2022 19:32
@changhiskhan changhiskhan merged commit 4832ad8 into main Aug 5, 2022
@changhiskhan changhiskhan deleted the changhiskhan/restore-thirdparty branch August 5, 2022 20:33
eddyxu pushed a commit that referenced this pull request Aug 6, 2022
For local development it's much faster to be able do python setup.py develop
So we use the previous thirdparty build script to build pyarrow against
system libarrow that we installed to enable liblance build.

The main change in this PR is changing the arrow ref being checked out. By
default this builds pyarrow 8.0.0.devXXX, which fails the pylance requirement
of pyarrow>=8,<9. Instead we use the 8.0.1 tag in pyarrow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants