Skip to content

Commit

Permalink
[Breaking] Append _amrex to each plotfile Tool
Browse files Browse the repository at this point in the history
Packaging AMReX for Conda, we received feedback that our plotfile tool
names are too generic and can collide with other packages.

Thus, we propose to rename them with a common suffix.
  • Loading branch information
ax3l committed Oct 18, 2023
1 parent 7ee2912 commit 95a86c7
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions Tools/Plotfile/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,11 @@ foreach( _exe IN LISTS _exe_names)
if (AMReX_CUDA)
set_source_files_properties(${_exe}.cpp PROPERTIES LANGUAGE CUDA)
endif()

# Add suffix to each tool's name to make them unique when installed.
# This avoids potential collisions of names on user systems, e.g., in
# software packages (Spack/Conda/Debian/...).
set_target_properties(${_exe} PROPERTIES OUTPUT_NAME "${_exe}_amrex")
endforeach()


Expand Down

0 comments on commit 95a86c7

Please sign in to comment.