This is an archive of the discontinued LLVM Phabricator instance.

[lldb] [test] Skip lldb-mi test if LLDB_DISABLE_PYTHON is used
ClosedPublic

Authored by mgorny on Feb 9 2019, 6:18 AM.

Diff Detail

Event Timeline

mgorny created this revision.Feb 9 2019, 6:18 AM

Instead of adding REQUIRES: lldb-mi to every test, it would be better to create an appropriate lit.local.cfg in the lit/tools/lldb-mi folder. I think something like:

if not "lldb-mi" in config.available_features:
  config.unsupported = True

ought to do the trick.

mgorny updated this revision to Diff 186207.Feb 11 2019, 2:46 AM

Used lit.local.cfg.

labath accepted this revision.Feb 11 2019, 5:43 AM
This revision is now accepted and ready to land.Feb 11 2019, 5:43 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2019, 6:09 AM