Skip to content

Commit 5f1281c

Browse files
committed
ENH: Use improved cmake variable cmake with GENEX_INCLUDE_DIRS
Replace INCLUDE2_DIRS with GENEX_INCLUDE_DIRS, where GENEX referees to generator expression.
1 parent d8fcc45 commit 5f1281c

1 file changed

Lines changed: 18 additions & 14 deletions

File tree

CMake/ITKModuleMacros.cmake

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -199,28 +199,28 @@ macro(itk_module_impl)
199199
endif()
200200

201201
# Prepare include directories with generator expressions for use in targets
202-
set(${itk-module}_INCLUDE2_DIRS "")
202+
set(${itk-module}_GENEX_INCLUDE_DIRS "")
203203
foreach(_dir ${${itk-module}_INCLUDE_DIRS})
204-
list(APPEND ${itk-module}_INCLUDE2_DIRS "$<BUILD_INTERFACE:${_dir}>")
204+
list(APPEND ${itk-module}_GENEX_INCLUDE_DIRS "$<BUILD_INTERFACE:${_dir}>")
205205
endforeach()
206206
list(
207207
APPEND
208-
${itk-module}_INCLUDE2_DIRS
208+
${itk-module}_GENEX_INCLUDE_DIRS
209209
"$<INSTALL_INTERFACE:$<INSTALL_PREFIX>/${${itk-module}_INSTALL_INCLUDE_DIR}>"
210210
)
211211

212212
# Prepare system include directories with generator expressions
213-
set(${itk-module}_SYSTEM_INCLUDE2_DIRS "")
213+
set(${itk-module}_SYSTEM_GENEX_INCLUDE_DIRS "")
214214
if(${itk-module}_SYSTEM_INCLUDE_DIRS)
215215
foreach(_dir ${${itk-module}_SYSTEM_INCLUDE_DIRS})
216216
list(
217217
APPEND
218-
${itk-module}_SYSTEM_INCLUDE2_DIRS
218+
${itk-module}_SYSTEM_GENEX_INCLUDE_DIRS
219219
"$<BUILD_INTERFACE:${_dir}>"
220220
)
221221
list(
222222
APPEND
223-
${itk-module}_SYSTEM_INCLUDE2_DIRS
223+
${itk-module}_SYSTEM_GENEX_INCLUDE_DIRS
224224
"$<INSTALL_INTERFACE:${_dir}>"
225225
)
226226
endforeach()
@@ -343,13 +343,13 @@ macro(itk_module_impl)
343343
target_include_directories(
344344
${itk-module}Module
345345
INTERFACE
346-
${${itk-module}_INCLUDE2_DIRS}
346+
${${itk-module}_GENEX_INCLUDE_DIRS}
347347
)
348348
target_include_directories(
349349
${itk-module}Module
350350
SYSTEM
351351
INTERFACE
352-
${${itk-module}_SYSTEM_INCLUDE2_DIRS}
352+
${${itk-module}_SYSTEM_GENEX_INCLUDE_DIRS}
353353
)
354354

355355
# Link transitive dependencies (public + compile depends) through ${itk-module}Module interface
@@ -455,13 +455,13 @@ macro(itk_module_impl)
455455
"${itk-module-RUNTIME_LIBRARY_DIRS-build}"
456456
)
457457
set(itk-module-INCLUDE_DIRS "${itk-module-INCLUDE_DIRS-build}")
458-
# set itk-module-INCLUDE2_DIRS so that includes both install interface and build interface path in appropriate generator expressions
459-
set(itk-module-INCLUDE2_DIRS "")
458+
# set itk-module-GENEX_INCLUDE_DIRS so that includes both install interface and build interface path in appropriate generator expressions
459+
set(itk-module-GENEX_INCLUDE_DIRS "")
460460
foreach(_dir ${itk-module-INCLUDE_DIRS-build})
461-
list(APPEND itk-module-INCLUDE2_DIRS "$<BUILD_INTERFACE:${_dir}>")
461+
list(APPEND itk-module-GENEX_INCLUDE_DIRS "$<BUILD_INTERFACE:${_dir}>")
462462
endforeach()
463463
foreach(_dir ${itk-module-INCLUDE_DIRS-install2})
464-
list(APPEND itk-module-INCLUDE2_DIRS "${_dir}")
464+
list(APPEND itk-module-GENEX_INCLUDE_DIRS "${_dir}")
465465
endforeach()
466466

467467
set(itk-module-EXPORT_CODE "${itk-module-EXPORT_CODE-build}")
@@ -721,14 +721,18 @@ macro(itk_module_add_library _name)
721721
target_compile_features(${_name} PUBLIC cxx_std_${CMAKE_CXX_STANDARD})
722722

723723
# Add module include directories to target
724-
target_include_directories(${_name} PUBLIC ${${itk-module}_INCLUDE2_DIRS})
724+
target_include_directories(
725+
${_name}
726+
PUBLIC
727+
${${itk-module}_GENEX_INCLUDE_DIRS}
728+
)
725729

726730
# Add module system include directories to target
727731
target_include_directories(
728732
${_name}
729733
SYSTEM
730734
PUBLIC
731-
${${itk-module}_SYSTEM_INCLUDE2_DIRS}
735+
${${itk-module}_SYSTEM_GENEX_INCLUDE_DIRS}
732736
)
733737

734738
# Add module library directories to target

0 commit comments

Comments
 (0)