[CMake] avoid complete recompilation and patchs problems (on mac for …

…instance) (medInria#781) * [Patchs] avoid compilation err applying patchs twice * [patches] build_always 0 for patches * [DCMTK] test ep_source_dir * [DCMTK] build_always 0 * [extproj] add if ep_source_dir for all * [CMake] rm ep_source_dir on master branches, mv BUILD_ALW in other PR Co-authored-by: ci <ci@music-macos1015.cs1cloud.internal> (cherry picked from commit acf3be2)
Florent2305 · Nov 4, 2020 · 3670f37 · 3670f37
1 parent 60f71f3
commit 3670f37
Show file tree

Hide file tree

Showing 11 changed files with 40 additions and 44 deletions.
diff --git a/superbuild/external_projects_tools/EP_GeneratePatchCommand.cmake b/superbuild/external_projects_tools/EP_GeneratePatchCommand.cmake
@@ -15,7 +15,7 @@ function(ep_GeneratePatchCommand ep OutVar)
         endif()
     endforeach()
 
-    set(PATCH_COMMAND)
+    set(PATCH_COMMAND "")
     if (NOT "${PATCHES_TO_APPLY}" STREQUAL "")
         set(PATCH_COMMAND ${GIT_BIN} apply --ignore-whitespace ${PATCHES_TO_APPLY})
     endif()

diff --git a/superbuild/projects_modules/DCMTK.cmake b/superbuild/projects_modules/DCMTK.cmake
@@ -36,11 +36,13 @@ EP_Initialisation(${ep}
 if (NOT USE_SYSTEM_${ep})
 
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
-set(git_url git://git.dcmtk.org/dcmtk.git)
-set(git_tag DCMTK-3.6.2)
+if (NOT DEFINED ${ep}_SOURCE_DIR)
+    set(git_url git://git.dcmtk.org/dcmtk.git)
+    set(git_tag DCMTK-3.6.2)
+endif()
 
 ## #############################################################################
 ## Check if patch has to be applied
@@ -104,7 +106,7 @@ ExternalProject_Add(${ep}
   BINARY_DIR ${build_path}
   TMP_DIR ${tmp_path}
   STAMP_DIR ${stamp_path}
-  
+
   GIT_REPOSITORY ${git_url}
   GIT_TAG ${git_tag}
   PATCH_COMMAND ${${ep}_PATCH_COMMAND}
@@ -123,7 +125,6 @@ ExternalProject_Add(${ep}
 ExternalProject_Get_Property(${ep} binary_dir)
 set(${ep}_DIR ${binary_dir} PARENT_SCOPE)
 
-
 endif() #NOT USE_SYSTEM_ep
 
 endfunction()
diff --git a/superbuild/projects_modules/ITK.cmake b/superbuild/projects_modules/ITK.cmake
@@ -35,11 +35,13 @@ EP_Initialisation(${ep}
 if (NOT USE_SYSTEM_${ep})
 
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
-set(git_url ${GITHUB_PREFIX}InsightSoftwareConsortium/ITK.git)
-set(git_tag v5.0.0)
+if (NOT DEFINED ${ep}_SOURCE_DIR)
+    set(git_url ${GITHUB_PREFIX}InsightSoftwareConsortium/ITK.git)
+    set(git_tag v5.0.0)
+endif()
 
 ## #############################################################################
 ## Add specific cmake arguments for configuration step of the project
@@ -99,16 +101,13 @@ ExternalProject_Add(${ep}
   BUILD_ALWAYS 1
   )
 
-
 ## #############################################################################
 ## Set variable to provide infos about the project
 ## #############################################################################
 
 ExternalProject_Get_Property(ITK binary_dir)
 set(${ep}_DIR ${binary_dir} PARENT_SCOPE)
 
-
-
 endif() #NOT USE_SYSTEM_ep
 
 endfunction()
diff --git a/superbuild/projects_modules/LogDemons.cmake b/superbuild/projects_modules/LogDemons.cmake
@@ -36,12 +36,11 @@ EP_Initialisation(${ep}
 
 if (NOT USE_SYSTEM_${ep})
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
-set(git_url ${GITHUB_PREFIX}Inria-Asclepios/LCC-LogDemons.git)
-set(git_tag master)
-
+  set(git_url ${GITHUB_PREFIX}Inria-Asclepios/LCC-LogDemons.git)
+  set(git_tag master)
 
 ## #############################################################################
 ## Add specific cmake arguments for configuration step of the project
@@ -92,10 +91,9 @@ ExternalProject_Add(${ep}
   CMAKE_ARGS ${cmake_args}
   DEPENDS ${${ep}_dependencies}
   INSTALL_COMMAND ""
-  BUILD_ALWAYS 1   
+  BUILD_ALWAYS 1
   )
 
-
 ## #############################################################################
 ## Set variable to provide infos about the project
 ## #############################################################################

diff --git a/superbuild/projects_modules/QtDCM.cmake b/superbuild/projects_modules/QtDCM.cmake
@@ -37,11 +37,11 @@ EP_Initialisation(${ep}
 if (NOT USE_SYSTEM_${ep})
 
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
 set(git_url ${GITHUB_PREFIX}medInria/qtdcm.git)
-set(git_tag APHP) 
+set(git_tag APHP)
 
 ## #############################################################################
 ## Add specific cmake arguments for configuration step of the project

diff --git a/superbuild/projects_modules/RPI.cmake b/superbuild/projects_modules/RPI.cmake
@@ -37,11 +37,13 @@ EP_Initialisation(${ep}
 if (NOT USE_SYSTEM_${ep})
 
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
-set(git_url ${GITHUB_PREFIX}Inria-Asclepios/RPI.git)
-set(git_tag ITK5.0)
+if (NOT DEFINED ${ep}_SOURCE_DIR)
+    set(git_url ${GITHUB_PREFIX}Inria-Asclepios/RPI.git)
+    set(git_tag ITK5.0)
+endif()
 
 ## #############################################################################
 ## Add specific cmake arguments for configuration step of the project
@@ -91,8 +93,7 @@ ExternalProject_Add(${ep}
   DEPENDS ${${ep}_dependencies}
   INSTALL_COMMAND ""
   BUILD_ALWAYS 1
-  ) 
-
+  )  
 
 ## #############################################################################
 ## Set variable to provide infos about the project

diff --git a/superbuild/projects_modules/TTK.cmake b/superbuild/projects_modules/TTK.cmake
@@ -35,14 +35,14 @@ EP_Initialisation(${ep}
   )
 
 if (NOT USE_SYSTEM_${ep})
+
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
 set(git_url ${GITHUB_PREFIX}Inria-Asclepios/TTK-Public.git)
 set(git_tag master)
 
-
 ## #############################################################################
 ## Add specific cmake arguments for configuration step of the project
 ## #############################################################################
@@ -91,17 +91,15 @@ ExternalProject_Add(${ep}
   CMAKE_ARGS ${cmake_args}
   DEPENDS ${${ep}_dependencies}
   INSTALL_COMMAND ""
-  BUILD_ALWAYS 1   
-  )
-
+  BUILD_ALWAYS 1
+  )  
 
 ## #############################################################################
 ## Set variable to provide infos about the project
 ## #############################################################################
 
 ExternalProject_Get_Property(${ep} binary_dir)
 set(${ep}_DIR ${binary_dir} PARENT_SCOPE)
-
 
 endif() #NOT USE_SYSTEM_ep
 

diff --git a/superbuild/projects_modules/VTK.cmake b/superbuild/projects_modules/VTK.cmake
@@ -35,11 +35,13 @@ EP_Initialisation(${ep}
 if (NOT USE_SYSTEM_${ep})
 
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
-set(git_url ${GITHUB_PREFIX}Kitware/VTK.git)
-set(git_tag v8.1.2)
+if (NOT DEFINED ${ep}_SOURCE_DIR)
+    set(git_url ${GITHUB_PREFIX}Kitware/VTK.git)
+    set(git_tag v8.1.2)
+endif()
 
 ## #############################################################################
 ## Add specific cmake arguments for configuration step of the project

diff --git a/superbuild/projects_modules/dtk.cmake b/superbuild/projects_modules/dtk.cmake
@@ -37,12 +37,13 @@ EP_Initialisation(${ep}
 if (NOT USE_SYSTEM_${ep})
 
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
-set(git_url ${GITLAB_INRIA_PREFIX}dtk/dtk.git)
-set(git_tag 1.7.1)
-
+if (NOT DEFINED ${ep}_SOURCE_DIR)
+    set(git_url ${GITLAB_INRIA_PREFIX}dtk/dtk.git)
+    set(git_tag 1.7.1)
+endif()
 
 ## #############################################################################
 ## Add specific cmake arguments for configuration step of the project
@@ -81,7 +82,6 @@ set(cmake_args
   -DQt5_DIR=${Qt5_DIR}
   )
 
-
 ## #############################################################################
 ## Add external-project
 ## #############################################################################
@@ -105,7 +105,6 @@ ExternalProject_Add(${ep}
   BUILD_ALWAYS 1
   )
 
-
 ## #############################################################################
 ## Set variable to provide infos about the project
 ## #############################################################################

diff --git a/superbuild/projects_modules/dtkImaging.cmake b/superbuild/projects_modules/dtkImaging.cmake
@@ -37,13 +37,12 @@ EP_Initialisation(${ep}
 if (NOT USE_SYSTEM_${ep})
 
 ## #############################################################################
-## Set up versioning control.
+## Set up versioning control
 ## #############################################################################
 
 set(git_url ${GITLAB_INRIA_PREFIX}dtk/dtk-imaging.git)
 set(git_tag master)
 
-
 ## #############################################################################
 ## Add specific cmake arguments for configuration step of the project
 ## #############################################################################
@@ -94,7 +93,6 @@ ExternalProject_Add(${ep}
   BUILD_ALWAYS 1
   )
 
-
 ## #############################################################################
 ## Set variable to provide infos about the project
 ## #############################################################################

diff --git a/superbuild/projects_modules/ffmpeg.cmake b/superbuild/projects_modules/ffmpeg.cmake
@@ -36,7 +36,7 @@ EP_Initialisation(${ep}
 if (NOT USE_SYSTEM_${ep})
 
 ## #############################################################################
-## Define repository where get the sources
+## Set up versioning control
 ## #############################################################################
 
 if (NOT DEFINED ${ep}_SOURCE_DIR)