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

Part 2 of #3933 failing make install arrow #3944

Merged

Conversation

ajpotts
Copy link
Contributor

@ajpotts ajpotts commented Dec 13, 2024

This add the ability for make install-arrow to install from source code from the git release of arrow.

Part 2 of #3933 failing make install arrow

@ajpotts ajpotts force-pushed the 3933_failing_make_install-arrow_pt2 branch 2 times, most recently from d7e4f81 to fff71f5 Compare December 13, 2024 19:10
@ajpotts ajpotts force-pushed the 3933_failing_make_install-arrow_pt2 branch from 78dd120 to 03fae26 Compare December 13, 2024 21:09
@ajpotts ajpotts marked this pull request as ready for review December 13, 2024 21:09
@ajpotts
Copy link
Contributor Author

ajpotts commented Dec 13, 2024

You may need to install build-essential, ninja-build, cmake to build with this option.

This can be tested with make install-arrow DEP_BUILD_DIR=<dir> OS=unrecognized.

@ajpotts
Copy link
Contributor Author

ajpotts commented Dec 13, 2024

Here is a relevant link to the arrow documentation: https://arrow.apache.org/docs/dev/developers/cpp/building.html#building-arrow-cpp

Copy link
Contributor

@drculhane drculhane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This worked as advertised on Alma LInux 8.

Worth noting: the package manager dnf didn't seem to know where to get either build-essential or ninja-build, so you probably don't need those. Also, cmake is already a prerequisite for chapel, so you probably don't need to spell that one out either.

@ajpotts ajpotts added this pull request to the merge queue Dec 16, 2024
Merged via the queue into Bears-R-Us:master with commit 0270c50 Dec 16, 2024
16 checks passed
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 16, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 16, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 16, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 17, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 18, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 26, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 27, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 27, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 27, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 27, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 28, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 30, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 30, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Dec 30, 2024
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 3, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 3, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 3, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 3, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 3, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 4, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 7, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 8, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 8, 2025
ajpotts added a commit to ajpotts/arkouda that referenced this pull request Jan 16, 2025
jabraham17 pushed a commit to jabraham17/arkouda that referenced this pull request Jan 21, 2025
jabraham17 pushed a commit to jabraham17/arkouda that referenced this pull request Jan 21, 2025
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