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

Fix arrow install with custom INSTALL_PREFIX #11052

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

czentgr
Copy link
Collaborator

@czentgr czentgr commented Sep 20, 2024

If the recently added INSTALL_PREFIX does not point to /usr/local in Linux then the bundled Thrift installation from the Arrow dependency fails because it cannot locate Boost.

Boost was previously installed into the INSTALL_PREFIX but the Thrift CMake in Arrow does not pass on any PREFIX_PATH settings to Thrift. The PREFIX_PATH is set to the INSTALL_PREFIX and works with Arrow itself just fine.

The error would be

Could NOT find Boost (missing: Boost_INCLUDE_DIR) (Required is at least
version "1.56")

If the recently added INSTALL_PREFIX does not point to /usr/local
in Linux then the bundled Thrift installation from the Arrow dependency
fails because it cannot locate Boost.

Boost was previously installed into the INSTALL_PREFIX but the Thrift CMake in Arrow
does not pass on any PREFIX_PATH settings to Thrift. The PREFIX_PATH is set to the
INSTALL_PREFIX and works with Arrow itself just fine.

The error would be

Could NOT find Boost (missing: Boost_INCLUDE_DIR) (Required is at least
  version "1.56")

Co-authored-by: Majeti Deepak <deepak.majeti@ibm.com>
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 20, 2024
Copy link

netlify bot commented Sep 20, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit e163a98
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/66ede74b9540640008a2d706

@czentgr czentgr marked this pull request as ready for review September 21, 2024 02:25
@czentgr
Copy link
Collaborator Author

czentgr commented Sep 21, 2024

@majetideepak FYI

@zuyu
Copy link
Contributor

zuyu commented Sep 21, 2024

@czentgr I think #11059 also fixes the Arrow build. So, we might not need the fix in this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants