arphaman added inline comments.
================ Comment at: test/Tooling/clang-diff-json.cpp:1 +// RUN: clang-diff -ast-dump %s -- | python -m json.tool | FileCheck %s + ---------------- arphaman wrote: > johannes wrote: > > johannes wrote: > > > arphaman wrote: > > > > I think you have to use `%python` instead of `python`, like LLVM tests > > > > do. > > > ok > > > So I have to use python 2.7? (for the test in > > > https://reviews.llvm.org/D36182) > > It looks like %python is expanded in the wrong way. > > I get the same as when using %Sython, the directory of the test case is > > substituted > Yeah, 2.7 support is still a requirement. It might not be in the lit.cfg for Clang, let me check. https://reviews.llvm.org/D36178 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
