Re: [Lldb-commits] [PATCH] D13577: Fix build with python disabled after r249597

This revision was automatically updated to reflect the committed changes. Closed by commit rL250494: Fix build with python disabled after r249597 (authored by kfischer). Changed prior to commit: http://reviews.llvm.org/D13577?vs=36918&id=37557#toc Repository: rL LLVM http://reviews.llvm.org

Re: [Lldb-commits] [PATCH] D13577: Fix build with python disabled after r249597

granata.enrico accepted this revision. granata.enrico added a comment. Looks fine to me Repository: rL LLVM http://reviews.llvm.org/D13577 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/ll

Re: [Lldb-commits] [PATCH] D13577: Fix build with python disabled after r249597

clayborg accepted this revision. clayborg added a comment. Wait for the OK from Enrico as well to verify this is OK. Repository: rL LLVM http://reviews.llvm.org/D13577 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.or

Re: [Lldb-commits] [PATCH] D13577: Fix build with python disabled after r249597

evgeny777 accepted this revision. evgeny777 added a comment. This revision is now accepted and ready to land. Lgtm Repository: rL LLVM http://reviews.llvm.org/D13577 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/

Re: [Lldb-commits] [PATCH] D13577: Fix build with python disabled after r249597

brucem added a subscriber: brucem. brucem added a comment. Getting rid of a lot of the `#ifndef LLDB_DISABLE_PYTHON` checks had been on my list of things I wanted to clean up. Since you regularly build with this configuration (and I never do), I suspect that a lot more of those checks could go