[Lldb-commits] [lldb] [lldb][windows] fix install-lldb target on Windows (PR #196070)

[David Spickett via lldb-commits]

DavidSpickett wrote:

Github is being its usual stable, working, self, so here's the comment I was going to post on the diff:
Whats the effect of the `if(LLDB_ENABLE_... AND TARGET ...)?

I see that we should be only doing this if the targets exist, because we need to make dependencies. However, I'm not sure that we need both parts of the `if`.

Is there a situation where LLDB_ENABLE_... is off, but the targets get made anyway they're just not used?

Also this fixes a Windows issue but the fix seems quite generic. Has this also been an issue for Linux and we've not noticed it? 

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