Skip to content

Conversation

@thisisnic
Copy link
Contributor

No description provided.

@jeroen
Copy link
Contributor

jeroen commented May 4, 2023

I'll have a look at this tomorrow, the deployments urls have changed a bit so I need to update this accordingly.

@jeroen
Copy link
Contributor

jeroen commented May 5, 2023

FYI, one problem with this PR is that if you add the -larrow_acero and -DARROW_R_WITH_ACERO flags unconditionally, this breaks the build for versions of arrow before 12 which did not have libarrow_acero.

@thisisnic
Copy link
Contributor Author

FYI, one problem with this PR is that if you add the -larrow_acero and -DARROW_R_WITH_ACERO flags unconditionally, this breaks the build for versions of arrow before 12 which did not have libarrow_acero.

Thanks for fixing that!

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