[Lldb-commits] [PATCH] D158022: [lldb] Print an actionable error message when sphinx_automodapi is not installed

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere added a comment.

In D158022#4590291 <https://reviews.llvm.org/D158022#4590291>, @mib wrote:

> We should really add a `requirements.txt` file with all the dependencies.

I've long wanted to make this all automated, and have CMake install all the necessary dependencies. But that seems a bit intrusive without using something like a virtual environment, which would have to be project-wide and more work than I'm willing to put in. Regardless, that wouldn't actually help here, at least not on macOS if you installed sphinx with homebrew, because it packages its own version of Python (`/opt/homebrew/Cellar/sphinx-doc/7.1.2/libexec/bin/python`) and you have to install the `sphinx_automodapi` module with that. That's why I print `sys.executable` and really the motivation behind this patch.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158022/new/

https://reviews.llvm.org/D158022