Remove undefined ASAN_RUNTIME_DEP from runtime deps - #10
Conversation
Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>
|
@copilot ASAN_RUNTIME_DEP is already removed and pushed to feature/mimalloc branch. You mentioned about adding $ to make the generator expression clearer in PR #6 "$<IF:$AND:$BOOL:${USE_MIMALLOC},$CONFIG:Release,${MIMALLOC_INSTALL_DIR}/bin/mimalloc.dll;${MIMALLOC_INSTALL_DIR}/bin/mimalloc-redirect.dll,> |
I've verified that |
|
@copilot I was referring to including
|
Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>
* Add optional mimalloc allocator support for Release builds. * Update RoiManager/CMakeLists.txt to check mimalloc target exists before linking. Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update PluginManager/CMakeLists.txt to check mimalloc target exists before linking. Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update CMakeLists.txt to show message that mimalloc will be linked for Release only. Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Added link option /INCLUDE:mi_version in cvutil/CMakeLists.txt only for MSVC Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Updated CMakeLists.txt to include platform specific import locations. Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix generator expression in ExternalProject_Add for mimalloc (#8) * Initial plan * Replace $<CONFIG> with ${CMAKE_BUILD_TYPE} in ExternalProject_Add Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com> * Use hardcoded Release build type for mimalloc ExternalProject Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com> * Remove undefined ASAN_RUNTIME_DEP from runtime deps (#9) * Initial plan * Remove undefined ASAN_RUNTIME_DEP from EXTRA_RUNTIME_DEPS Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com> * Remove undefined ASAN_RUNTIME_DEP from runtime deps (#10) * Initial plan * Remove undefined ASAN_RUNTIME_DEP from EXTRA_RUNTIME_DEPS expression Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com> * Use $<IF> syntax for EXTRA_RUNTIME_DEPS generator expression Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com> --------- Co-authored-by: Seethepalli <7uz@ornl.gov> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Addresses feedback from #6 regarding undefined
ASAN_RUNTIME_DEPvariable reference in the EXTRA_RUNTIME_DEPS CMake expression.Changes
$<$<CONFIG:Debug>:;${ASAN_RUNTIME_DEP}>from the generator expression since the variable is never defined$<IF>syntax for cleaner generator expressionBefore
After
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.