Merge pull request #691 from borglab/fix-uninstall-target-exists

Avoid target collision if gtsam used as submodule
borglab · Mar 8, 2021 · 48232a3 · 48232a3
2 parents 7f06add + ffc4e59
commit 48232a3
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/cmake/HandleUninstall.cmake b/cmake/HandleUninstall.cmake
@@ -6,5 +6,11 @@ configure_file(
   "${CMAKE_CURRENT_BINARY_DIR}/cmake_uninstall.cmake"
   IMMEDIATE @ONLY)
 
-add_custom_target(uninstall
-  "${CMAKE_COMMAND}" -P "${CMAKE_CURRENT_BINARY_DIR}/cmake_uninstall.cmake")
+if (NOT TARGET uninstall) # avoid duplicating this target
+  add_custom_target(uninstall
+    "${CMAKE_COMMAND}" -P "${CMAKE_CURRENT_BINARY_DIR}/cmake_uninstall.cmake")
+else()
+    add_custom_target(uninstall_gtsam
+      "${CMAKE_COMMAND}" -P "${CMAKE_CURRENT_BINARY_DIR}/cmake_uninstall.cmake")
+    add_dependencies(uninstall uninstall_gtsam)
+endif()