On 5/21/24 09:33, Vitaly Zaitsev via devel wrote:
On 27/04/2024 06:34, Tom Stellard wrote:
If anyone has any feedback on these ideas we'd like to hear it and are happy to 
discuss
these more.

Can you fix the ${LLVM_LIBRARY_DIR} variable in LLVM's CMake config?

It broke after switching LLVM to use /usr/lib. It still points to /usr/lib64 
and we need to patch it manually[1].


I looked in the iwyu source, and it's not using ${LLVM_LIBRARY_DIR} correctly.
That variable points to where the libraries are installed, but iwyu is using it 
to look
up the resource directory.  iwyu should be using `clang -print-resource-dir`
instead.

-Tom

[1]: https://src.fedoraproject.org/rpms/iwyu/blob/rawhide/f/iwyu.spec#_51

--
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to