From 6006b5e9e9740212c3fed97ad456eb00c4204035 Mon Sep 17 00:00:00 2001 From: Ronald Caesar Date: Fri, 16 Jan 2026 23:23:22 -0400 Subject: [PATCH] build: mark doc target as ALL_BUILD Windows does not support the doc target, which causes the CI to fail. Marking the doc target as ALL should fix this. Signed-off-by: Ronald Caesar --- .github/workflows/build.yml | 6 ++---- CMakeLists.txt | 2 +- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 333ca6d..be5882e 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -132,9 +132,8 @@ jobs: -DBALLISTIC_ENABLE_BUILD_TESTS=ON \ -DBALLISTIC_ENABLE_LINK_TIME_OPTIMIZATION=${{ matrix.type == 'Release' && 'ON' || 'OFF' }} - - name: Build Project & Docs - # FIX: Replaced env var with relative path - run: cmake --build build --config ${{ matrix.type }} --target all doc --parallel 4 + - name: Build Project + run: cmake --build build --config ${{ matrix.type }} --target all --parallel 4 - name: Run Test Suite working-directory: build @@ -145,7 +144,6 @@ jobs: uses: actions/upload-artifact@v4 with: name: ballistic-tools-${{ matrix.os }} - # Relative paths are safe in action inputs path: | build/ballistic_cli* build/decoder_cli* diff --git a/CMakeLists.txt b/CMakeLists.txt index e1794b7..00f2ffd 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -106,7 +106,7 @@ if(CMAKE_SYSTEM_NAME STREQUAL "Linux" OR CMAKE_SYSTEM_NAME STREQUAL "Darwin") set (PROJECT_HEADERS include/bal_engine.h include/bal_decoder.h include/bal_memory.h include/bal_types.h include/bal_errors.h) - add_custom_target(doc + add_custom_target(doc ALL COMMAND ${CMAKE_CURRENT_BINARY_DIR}/cdoc docs/cdoc ${PROJECT_HEADERS} WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} COMMENT "Generating Documentation..."