Skip to content

Commit

Permalink
Merge pull request #265 from DUNE-DAQ/johnfreeman/issue262_non_terrib…
Browse files Browse the repository at this point in the history
…le_warning_message

JCF: Issue #262: give actionable advice to developers when their pack…
  • Loading branch information
andrewmogan authored Mar 14, 2024
2 parents 06cef8a + 801c07a commit de4f8b3
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions cmake/DAQTopprojHelpers.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ macro(daq_add_subpackages build_order)

# Warn the user that the build order of some package is not known
foreach(pkg ${found_pkgs})
message(WARNING "Package \"${pkg}\" not provided to the daq_add_subpackages function in ${CMAKE_CURRENT_SOURCE_DIR}/CMakeLists.txt. ${pkg} will only be built after the provided packages are built. There's no guarantee this is the correct order; if you know where ${pkg} should be in the dependency hierarchy add it to its appropriate place in the list of packages provided to daq_add_subpackages.")
message(WARNING "Package \"${pkg}\" not provided to the daq_add_subpackages function in ${CMAKE_CURRENT_SOURCE_DIR}/CMakeLists.txt. ${pkg} will only be built after the provided packages are built. There's no guarantee this is the correct order; if you know where ${pkg} should be in the dependency hierarchy add it to its appropriate place in ${CMAKE_CURRENT_SOURCE_DIR}/dbt-build-order.cmake")
endforeach()

set(pkgs ${known_pkgs} ${found_pkgs})
Expand All @@ -138,4 +138,5 @@ macro(daq_add_subpackages build_order)

daq_topproj_restore_gnudirs()

endmacro()
endmacro()

0 comments on commit de4f8b3

Please sign in to comment.