[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