[Lldb-commits] [PATCH] D16150: Fix project name conflict for MSBuild

[Jonathan Meier via lldb-commits]

jonathanmeier created this revision.
jonathanmeier added a reviewer: zturner.
jonathanmeier added a subscriber: lldb-commits.

CMake allows case sensitive folder naming when setting the folder property of a target. However MSBuild doesn't like that. It complains about having two projects named the same when trying to compile the solution and aborts. (Solution file error MSB5004: The solution file has two projects named "lldb tests")

In lldb/unittests/CMakeLists.txt a folder "LLDB tests" is defined and in lldb/lit/CMakeLists.txt a folder "lldb tests" is defined.

Renaming "lldb tests" to "LLDB tests" in lldb/lit/CMakeLists.txt fixes the problem.

http://reviews.llvm.org/D16150

Files:
  lit/CMakeLists.txt

Index: lit/CMakeLists.txt
===================================================================
--- lit/CMakeLists.txt
+++ lit/CMakeLists.txt
@@ -33,4 +33,4 @@
   DEPENDS ${LLDB_TEST_DEPS}
   )
 
-set_target_properties(check-lldb-unit PROPERTIES FOLDER "lldb tests")
+set_target_properties(check-lldb-unit PROPERTIES FOLDER "LLDB tests")


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16150.44775.patch
Type: text/x-patch
Size: 336 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20160113/5294c04e/attachment.bin>