Skip to content

Commit

Permalink
Fix cmake dependency for moved test runner (#4440)
Browse files Browse the repository at this point in the history
When I moved and renamed the test runner, I neglected to update the cmake target executable name.  This means the runner does in fact get built the first time you run `make`, but it won't get incrementally updated if sources change.

This PR updates the name of the executable so cmake can correctly track it.
  • Loading branch information
chalcolith authored Sep 8, 2023
1 parent 46eb348 commit 05c75da
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/full-program-runner/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
project(libponyc.tests.runner)

set(RUNNER_EXECUTABLE "runner${CMAKE_EXECUTABLE_SUFFIX}")
set(RUNNER_EXECUTABLE "full-program-runner${CMAKE_EXECUTABLE_SUFFIX}")

add_custom_target(tests.runner ALL DEPENDS ${RUNNER_EXECUTABLE})

Expand Down

0 comments on commit 05c75da

Please sign in to comment.