Skip to content

Commit 624d30c

Browse files
authored
cmake: fix includes for dictionary generation (#176)
See root-project/root#8308, where the issue is rightly described as 'counter-intuitive'
1 parent b18ec3a commit 624d30c

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

analyzers/dataframe/CMakeLists.txt

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -36,15 +36,11 @@ if(NOT ${WITH_DD4HEP})
3636
list(FILTER sources EXCLUDE REGEX "CaloNtupleizer.cc")
3737
endif()
3838

39-
ROOT_GENERATE_DICTIONARY(G__FCCAnalyses
40-
${headers}
41-
LINKDEF include/FCCAnalyses/LinkDef.h
42-
)
4339

4440
message(STATUS "CMAKE_CURRENT_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}")
4541
message(STATUS "CMAKE_INSTALL_INCLUDEDIR ${CMAKE_INSTALL_INCLUDEDIR}")
4642

47-
add_library(FCCAnalyses SHARED ${sources} ${headers} G__FCCAnalyses.cxx )
43+
add_library(FCCAnalyses SHARED ${sources} ${headers} )
4844
target_include_directories(FCCAnalyses PUBLIC
4945
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
5046
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>
@@ -82,6 +78,12 @@ endif()
8278
set_target_properties(FCCAnalyses PROPERTIES
8379
PUBLIC_HEADER "${headers}")
8480

81+
ROOT_GENERATE_DICTIONARY(G__FCCAnalyses
82+
${headers}
83+
MODULE FCCAnalyses
84+
LINKDEF include/FCCAnalyses/LinkDef.h
85+
)
86+
8587
install(TARGETS FCCAnalyses
8688
EXPORT FCCAnalysesTargets
8789
RUNTIME DESTINATION "${INSTALL_BIN_DIR}" COMPONENT bin

0 commit comments

Comments
 (0)