Add macro for uninstall target and fix symlink tracking#4775
Add macro for uninstall target and fix symlink tracking#4775jmsexton03 wants to merge 2 commits intoAMReX-Codes:developmentfrom
Conversation
|
Thank you for this PR and sorry for the long wait. There is a small problem: the install prefix now lacks The symlink install logic around |
|
Thanks again for putting this together, and sorry this sat for so long. After looking at a recent PR in #5277, one can fold the symlink/install-manifest bug fix in there, mainly because that PR was already the narrower diff for the existing The legacy link is now also recorded in If you still want the reusable uninstall target machinery, that could still make sense to add (e.g. update this PR?). This is useful on its own and independent from the symlink bug. |
|
updated PR accordingly :) does that work? |
Summary
add_uninstall_targetmacro to create reusable uninstall targetTrack libamrex symlink in install manifest for proper cleanup:done via Create symbolic link during install #5277AMReX_cmake_uninstall.intemplate for uninstall scriptAdditional background
Previously, the libamrex symlink was not tracked in the install manifest. Newer CMake versions provide better ways to reference source locations using variables instead of hardcoded paths, allowing manifest information to be stored more accurately. The new macro also prevents conflicts when multiple projects using AMReX each try to create their own uninstall targets.
Checklist
The proposed changes: