This enables build on NetBSD-7.0. It would be nice to have a fall-back for
externally provided ncurses via pkgsrc, but it's left after the scheduled drop
of the autoconf/gmake build system. For not it's not worth the effort.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
I don't approve of automagic dependencies. With this patch it won't be possible to force curses usage. Even if you specify -DLLDB_DISABLE_CURSES=0, it will still build (sure, it will show a message about not being found, but who reads those...) even if it can't find the library, leading to surprises further down the line. I recommend doing one of the following:
- Leave things as they are. It is already possible to disable curses with the right switch
- Tweak the error message to inform the user of possible actions (but leave it a fatal error). E.g., "Curses library not found. Please install the library or reconfigure with LLDB_DISABLE_CURSES=0."
- set LLDB_DISABLE_CURSES=1 on netbsd only. (less magic, it only depends on the platform, not it's runtime properties, which can change randomly)
Comment Actions
I resign from this change. I will just go for -DLLDB_DISABLE_CURSES from command line on NetBSD-7.0.