Respect CMAKE_DEBUG_POSTFIX setup

TheLartians · Jul 31, 2023 · 0bb3325 · 0bb3325
1 parent d81a424
commit 0bb3325
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/.github/workflows/standalone.yml b/.github/workflows/standalone.yml
@@ -32,5 +32,7 @@ jobs:
       - name: build
         run: cmake --build build -j4
 
-      - name: run
-        run: ./build/Greeter
+      - name: test
+        run: |
+          cd build
+          ctest --build-config Debug
diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml
@@ -28,7 +28,7 @@ jobs:
           key: ${{ github.workflow }}-cpm-modules-${{ hashFiles('**/CMakeLists.txt', '**/*.cmake') }}
 
       - name: configure
-        run: cmake -S test -B build -G "Visual Studio 17 2022" -D BUILD_SHARED_LIBS=NO
+        run: cmake -S test -B build  #  -G "Visual Studio 17 2022" -D BUILD_SHARED_LIBS=NO
 
       - name: build
         run: cmake --build build --config Debug -j4

diff --git a/standalone/CMakeLists.txt b/standalone/CMakeLists.txt
@@ -43,9 +43,6 @@ endif()
 target_link_libraries(${PROJECT_NAME} Greeter::Greeter cxxopts::cxxopts)
 set_target_properties(${PROJECT_NAME} PROPERTIES UNITY_BUILD ${OPTION_ENABLE_UNITY})
 
-# TODO(CK): why is this used? This overrides the CMAKE_DEBUG_POSTFIX!
-set_target_properties(${PROJECT_NAME} PROPERTIES OUTPUT_NAME Greeter)
-
 # --- Test it ---
 
 enable_testing()