rupprecht added a comment. In D70622#1771601 <https://reviews.llvm.org/D70622#1771601>, @rupprecht wrote:
> In D70622#1765674 <https://reviews.llvm.org/D70622#1765674>, @JDevlieghere > wrote: > > > No objections to moving the test. Jordan, let me know if you're up for it, > > otherwise I'm happy to take care of it. > > > I tried moving it today. The rabbit hole goes deep :D > The test framework seems pretty tied to the directory layout (because > python), e.g. those must be in a directory called "lldbsuite" to be able to > "import lldbsuite" > The test infrastructure also seems to assume tests + test framework are in > the same tree (e.g. the LLDB_TEST env var is used for both), so moving the > individual test directories and leaving just the framework classes in > lldbsuite breaks a few things. > > I agree it's probably cleaner to move the tests, so I'll keep poking. > Suggestions welcome though. I have it down to ~5 failing tests (some may be preexisting failures), so I'll start pulling apart my change into submittable pieces. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70622/new/ https://reviews.llvm.org/D70622 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
