Skip to content

Commit

Permalink
temporarily permit cmake dev warnings under CI
Browse files Browse the repository at this point in the history
... until upstream CPM fixes this warning:

    cpm-cmake/CPM.cmake#570
  • Loading branch information
Eric Woo committed Jul 15, 2024
1 parent 41a3b5b commit cd00607
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,9 @@ image: archlinux/archlinux:base-devel
.build_dependencies: &build_dependencies
- pacman --noconfirm -Sy cmake clang curl duktape pcre2

# TODO: add -Werror=dev back: https://github.com/cpm-cmake/CPM.cmake/pull/570
.build_steps: &build_steps
- cmake -Wdev -Werror=dev -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=1 . -B ./build
- cmake -Wdev -Wno-error=dev -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=1 . -B ./build
- cmake --build ./build
- cd ./build/tests && ctest -E '^cli-try-sandbox$' .

Expand All @@ -26,10 +27,11 @@ build_with_clang:
- export CXX=clang++
- *build_steps

# TODO: add -Werror=dev back: https://github.com/cpm-cmake/CPM.cmake/pull/570
format:
stage: build
before_script:
- *build_dependencies
script:
- cmake -Wdev -Werror=dev -DCMAKE_BUILD_TYPE=Debug . -B ./build
- cmake -Wdev -Wno-error=dev -DCMAKE_BUILD_TYPE=Debug . -B ./build
- cmake --build ./build --target fmt
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ add_executable(youtube-unthrottle)
target_sources(youtube-unthrottle PUBLIC main.c)
target_link_libraries(youtube-unthrottle PRIVATE sanitizers youtube my_cflags)

include(get_cpm) # TODO: https://github.com/cpm-cmake/CPM.cmake/pull/570
include(get_cpm)

add_subdirectory(src)
if (BUILD_TESTING)
Expand Down

0 comments on commit cd00607

Please sign in to comment.