[Lldb-commits] [PATCH] D53989: Fix formatting of wchar, char16, and char32
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Sat Nov 3 03:11:17 PDT 2018
labath added a comment.
lldb-server is used on architectures that don't have python (readily) available, and it uses the same codebase as the rest of lldb. (Granted, it only needs a small subset of that codebase, and this subset doesn't/shouldn't care about python, but we aren't able to split out that part cleanly (yet)). So I don't think requiring python is a good idea.
What we could do is make it a build-time error if we failed to detect&configure python *AND* the user hasn't explicitly disabled it. This would catch this problem early and avoid other surprises later on (e.g. the entire dotest test suite only works with python enabled. without it, you'd likely get some weird error).
https://reviews.llvm.org/D53989
More information about the lldb-commits
mailing list