Skip to content

Conversation

@fxcoudert
Copy link

Homebrew now ships LLVM 11. This prevents us rebuilding arrow from source:

CMake Warning at cmake_modules/FindLLVMAlt.cmake:27 (find_package):
  Could not find a configuration file for package "LLVM" that is compatible
  with requested version "10".

@github-actions
Copy link

Thanks for opening a pull request!

Could you open an issue for this pull request on JIRA?
https://issues.apache.org/jira/browse/ARROW

Then could you also rename pull request title in the following format?

ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}

See also:

@terencehonles
Copy link
Contributor

It looks like there are more changes needed since the builder is still broken https://github.com/apache/arrow/pull/8492/checks?check_run_id=1279869182

@terencehonles
Copy link
Contributor

Following up after the discussion in Homebrew/homebrew-core#63177 (comment) it looks like supporting LLVM 11 would be a good idea in the long term it is no longer a blocker to compiling arrow from source.

(I was able to fix this for the GitHub workflow by updating the cpp/Brewfile to only reference llvm@8 instead of llvm@8 and llvm)

@kou
Copy link
Member

kou commented Oct 21, 2020

@fxcoudert Thanks for sending feedback from Homebrew!

I think that we need more changes to support LLVM 11 like we did by 6a1cdbc for LLVM 10.

It may be difficult for people who don't familiar with Apache Arrow build system. I'll take over this.

@pprudhvi
Copy link
Contributor

@kou looks like this requires code changes, will we be able to build gandiva on centos with llvm 8?

@kou
Copy link
Member

kou commented Oct 22, 2020

will we be able to build gandiva on centos with llvm 8?

I think so.

@kou
Copy link
Member

kou commented Oct 24, 2020

#8521

@kou kou closed this Oct 24, 2020
@fxcoudert fxcoudert deleted the patch-1 branch October 24, 2020 19:20
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.

4 participants