[Lldb-commits] [lldb] [lldb] Add early CMake check for 'make' tool (PR #111531)

Stefan Gränitz via lldb-commits lldb-commits at lists.llvm.org
Thu Oct 10 03:25:00 PDT 2024


================
@@ -272,6 +272,8 @@ def parseOptionsAndInitTestdirs():
         configuration.make_path = "gmake"
     else:
         configuration.make_path = "make"
+    if ' ' in configuration.make_path:
+        configuration.make_path = f'"{configuration.make_path}"'
----------------
weliveindetail wrote:

I added this because yesterday my tests failed with:
```
'C:/Program' is not recognized as an internal or external command,
operable program or batch file.
```

I double-checked again today and this doesn't happen anymore. I am not sure why, maybe I had an inconsistent configuration state. The parameter for dotest.py is `--make C:/Program Files (x86)/GnuWin32/bin/make.exe` and the command that lldbtest.py runs is starting with `C:/Program Files (x86)/GnuWin32/bin/make.exe VPATH=...`. I cleared the cached in `lldb-test-build.noindex` and checked sure the binaries are recreated successfully by the test suite. It works without the quotes.

I am attaching a log file [TestBreakpointByLineAndColumn.log](https://github.com/user-attachments/files/17326545/TestBreakpointByLineAndColumn.log) with the commands dumped for future reference. I will the quoting and assume it just keeps working.

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


More information about the lldb-commits mailing list