hubert.reinterpretcast added inline comments.
================
Comment at: llvm/utils/lit/lit/LitConfig.py:98
+ self.fatal("Setting a timeout per test requires the"
+ " Python psutil module but it could not be"
+ " found. Try installing it via pip or via"
----------------
Minor nit: The quote indentation no longer lines up with the previous line.
================
Comment at: llvm/utils/lit/tests/lit.cfg:62
- lit_config.note('Found python psutil module')
- config.available_features.add("python-psutil")
except ImportError:
----------------
Removing `python-psutil` as a feature entirely may be a bit aggressive. It has
the potential of quietly disabling "out-of-tree" tests. I'm not sure that a
Phabricator patch about AIX has the right level of visibility for making such a
change. Can you send an RFC about the cleanup to the mailing list?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64251/new/
https://reviews.llvm.org/D64251
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
