Skip to content

Commit

Permalink
Merge pull request #1746 from DGtal-team/uninstall
Browse files Browse the repository at this point in the history
New option to disable the uninstall target
  • Loading branch information
dcoeurjo authored Oct 3, 2024
2 parents 90153d1 + 0713a0a commit 4e9fc6a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
2 changes: 2 additions & 0 deletions ChangeLog.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@

- *General*
- Fix cmake CGAL 6.0 Breaking change. (David Coeurjolly, [#1745](https:/DGtal-team/DGtal/pull/1745))
- Adding a new `DGTAL_REMOVE_UNINSTALL` cmake option to disable the `uninstall` target. (David Coeurjolly, [#1746](https:/DGtal-team/DGtal/pull/1746)


- *Geometry*
- Bug fix in ArithmeticalDSSComputerOnSurfels (Tristan Roussillon, [#1742](https:/DGtal-team/DGtal/pull/1742))
Expand Down
3 changes: 3 additions & 0 deletions cmake/Common.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -81,12 +81,15 @@ include(TargetDoxygenDox OPTIONAL)
# -----------------------------------------------------------------------------
# uninstall target
# -----------------------------------------------------------------------------
option(DGTAL_REMOVE_UNINSTALL "Remove DGtal uninstall target." OFF)
if (NOT DGTAL_REMOVE_UNINSTALL)
configure_file(
${CMAKE_CURRENT_SOURCE_DIR}/cmake/TargetUninstall.cmake.in
${CMAKE_CURRENT_BINARY_DIR}/TargetUninstall.cmake
@ONLY)
add_custom_target(uninstall
"${CMAKE_COMMAND}" -P "${CMAKE_CURRENT_BINARY_DIR}/TargetUninstall.cmake")
endif()

# -----------------------------------------------------------------------------
# Parsing cmake options
Expand Down

0 comments on commit 4e9fc6a

Please sign in to comment.