[Lldb-commits] [lldb] [lldb] Fix finding make tool for tests (PR #111980)

via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 11 13:14:45 PDT 2024


zeroomega wrote:

Hi,

I noticed that our LLDB Windows builders all failed after this change is landed:

```
CMake Error at C:/b/s/w/ir/x/w/llvm-llvm-project/lldb/test/API/CMakeLists.txt:60 (message):
   LLDB tests require 'make' tool.  Please pass via `LLDB_TEST_MAKE` (or
   otherwise disable tests with `LLDB_INCLUDE_TESTS=OFF`)
```

We are aware our windows bots don't have make tool and we are not planning to add it anyway. Not all lldb tests require `make` to run and it feels wrong to me to mandate `make` tool for a few tests that requires it. Could it be changed to disable such tests when make is not present instead of throw an error to prevent the lldb build?

https://github.com/llvm/llvm-project/pull/111980


More information about the lldb-commits mailing list