[Lldb-commits] [PATCH] D56741: [CMake] Explicitly list User32 as dependency of lldb-mi

Tue Jan 15 15:31:17 PST 2019

xiaobai added a comment.

In D56741#1358804 <https://reviews.llvm.org/D56741#1358804>, @zturner wrote:

> It seems like a pretty bad idea to have lldb-mi show a dialog box :-/
>
> I don't actually work on or have anything to do with lldb-mi though, but I don't know how well-maintained or supported it is these days anyway.  But if nobody chimes in and says why this is important to have, I would just assume delete the line.

It looks like this functionality has been available since lldb-mi was committed. I'd rather fix the build than modify the functionality not knowing if somebody actually values this behavior. Based on commit logs it looks like there is some amount of work that was done on lldb-mi last year, but I'm not really sure how much people care about lldb-mi honestly.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56741/new/

https://reviews.llvm.org/D56741