This is an archive of the discontinued LLVM Phabricator instance.

[lldb] fix building with panel.h being in /usr/include/ncurses/
ClosedPublic

Authored by llunak on Aug 4 2020, 9:42 AM.

Details

Summary

My openSUSE 15.2 has /usr/include/curses.h as a symlink to /usr/include/ncurses/curses.h , but there's no such symlink for panel.h . Prefer using /usr/include/ncurses for the includes if they are found there by the CMake check.

Diff Detail

Event Timeline

llunak created this revision.Aug 4 2020, 9:42 AM
llunak requested review of this revision.Aug 4 2020, 9:42 AM

I am not a cmake expert. Might be a good idea to check the git log for people with more cmake expertise?

mgorny accepted this revision.Aug 4 2020, 10:32 PM

I suppose this makes sense. Testing it on some more systems might be good but I guess buildbot will take care of that.

This revision is now accepted and ready to land.Aug 4 2020, 10:32 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 5 2020, 12:51 AM