Page MenuHomePhabricator

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

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

Details

Summary

Skip running lldb-mi tests when Python support is disabled. This causes
lldb-mi to unconditionally fail, and therefore all the relevant tests
fail as well.

Diff Detail

Repository
rL LLVM

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