[Lldb-commits] [PATCH] D71232: [lldb/Lua] Add Boilerplate for a Lua Script Interpreter
Michał Górny via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Dec 10 03:18:07 PST 2019
mgorny added inline comments.
Comment at: lldb/cmake/modules/LLDBConfig.cmake:28
> I think this will tick off some bots (and people) because it means that the default configuration will not build unless one has (compatible?) lua installed. Though I don't really like that, the usual way to handle external dependencies in llvm is to detect their presence and automatically disable the relevant functionality.
> Now, that's not how things work in lldb right now, so it _may_ make sense to do the same for lua (though it also may make sense to port everything to the llvm style). However, the current lldb behavior has been a source of friction in the past and I suspect a fresh build error might reignite some of that.
> Anyway, you have been warned...
Fixing this is one of the things at the far end of my todo. If you could look into replacing the disable logic with something better, a lot of people would really be grateful.
CHANGES SINCE LAST ACTION
More information about the lldb-commits