This is an archive of the discontinued LLVM Phabricator instance.

Define LLDB_DISABLE_CURSES as a fallback for missing curses(3) or panel(3)
AbandonedPublic

Authored by krytarowski on Nov 16 2015, 8:21 PM.

Details

Summary

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.

Diff Detail

Repository
rL LLVM

Event Timeline

krytarowski retitled this revision from to Define LLDB_DISABLE_CURSES as a fallback for missing curses(3) or panel(3).
krytarowski updated this object.
krytarowski added reviewers: clayborg, emaste.
krytarowski set the repository for this revision to rL LLVM.
krytarowski added subscribers: lldb-commits, joerg, brucem.
labath requested changes to this revision.Nov 17 2015, 1:57 AM
labath added a reviewer: labath.
labath added a subscriber: labath.

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)
This revision now requires changes to proceed.Nov 17 2015, 1:57 AM
krytarowski abandoned this revision.Nov 17 2015, 3:28 AM

I resign from this change. I will just go for -DLLDB_DISABLE_CURSES from command line on NetBSD-7.0.