Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove CMake code once Axom and BLT are updated #245

Open
ldowen opened this issue Nov 9, 2023 · 0 comments
Open

Remove CMake code once Axom and BLT are updated #245

ldowen opened this issue Nov 9, 2023 · 0 comments
Assignees

Comments

@ldowen
Copy link
Collaborator

ldowen commented Nov 9, 2023

The code CMake code for bringing in Axom should be unnecessary in future updates. Specifically:

foreach(_comp ${AXOM_COMPONENTS_ENABLED})
list(APPEND SPHERAL_BLT_DEPENDS axom::${_comp})
get_target_property(axom_deps axom::${_comp} INTERFACE_LINK_LIBRARIES)
list(APPEND SPHERAL_BLT_DEPENDS ${axom_deps})
endforeach()

and

set(_props)
if( ${CMAKE_VERSION} VERSION_GREATER_EQUAL "3.13.0" )
list(APPEND _props INTERFACE_LINK_OPTIONS)
endif()
list(APPEND _props INTERFACE_COMPILE_OPTIONS)
foreach(_target axom axom::openmp)
if(TARGET ${_target})
message(STATUS "Removing OpenMP Flags from target[${_target}]")
foreach(_prop ${_props})
get_target_property(_flags ${_target} ${_prop})
if ( _flags )
string( REPLACE "${OpenMP_CXX_FLAGS}" ""
correct_flags "${_flags}" )
string( REPLACE "${OpenMP_Fortran_FLAGS}" ""
correct_flags "${correct_flags}" )
set_target_properties( ${_target} PROPERTIES ${_prop} "${correct_flags}" )
endif()
endforeach()
endif()
endforeach()

@ldowen ldowen self-assigned this Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant