kou commented on PR #14102: URL: https://github.com/apache/arrow/pull/14102#issuecomment-1248606756
> CMake is easily able to find it based on that, but the custom (AFAICT) `findLLVMAlt` has it's own checks that fail to find it now (or need something more; is `LLVM_DIR` suppqosed to be `$PREFIX/lib` or `$PREFIX/include/llvm` or something else?) Could you share the URL for the error? > But exposing such a large dependency as LLVM in headers should never have been accepted IMHO (especially as there's probably no reason to do so). Let's discuss this on `[email protected]` with Gandiva maintainers. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
