Skip to content

Commit

Permalink
Force target name to match the executable name
Browse files Browse the repository at this point in the history
I don't like this change, but if we don't do this the compiled binary
strangely breaks binary equivalence with the previous commit.
  • Loading branch information
SeanBryan51 committed Jan 19, 2024
1 parent b9bc2f3 commit 3f05aca
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -143,14 +143,12 @@ else()
set(CABLE_EXE_NAME cable)
endif()

add_executable(cable ${CABLE_SRCS})

set_target_properties(cable PROPERTIES OUTPUT_NAME ${CABLE_EXE_NAME})
add_executable(${CABLE_EXE_NAME} ${CABLE_SRCS})

# third party libs
target_link_libraries(
cable
${CABLE_EXE_NAME}
PkgConfig::NETCDF
)

install(TARGETS cable RUNTIME)
install(TARGETS ${CABLE_EXE_NAME} RUNTIME)

0 comments on commit 3f05aca

Please sign in to comment.