Skip to content

Commit

Permalink
Switch integration tests to GCC. For some reason only getting unexpec…
Browse files Browse the repository at this point in the history
…ted results in github actions.
  • Loading branch information
rayburgemeestre committed Dec 20, 2024
1 parent 99f8360 commit 6625620
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ jobs:
run: make test
- name: make download-integration-test-reference
run: make download-integration-test-reference
- name: make integration-test
run: make integration-test
- name: make integration-test-gcc
run: make integration-test-gcc
- name: upload integration test artifacts
if: always()
uses: actions/upload-artifact@v3
Expand Down
6 changes: 6 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,12 @@ integration-test: ## execute starcry unit tests using docker (with clang)
cmake --build build --target integration_tests -j $$(nproc) && \
./build/integration_tests -s -d yes --rng-seed 0)

.PHONY: integration-test-gcc
integration-test-gcc: ## execute starcry unit tests using docker (with gcc)
@$(call make, CMAKE_EXE_LINKER_FLAGS=-fuse-ld=mold CXX=$$(which c++) cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=1 -B build && \
cmake --build build --target integration_tests -j $$(nproc) && \
./build/integration_tests -s -d yes --rng-seed 0)

.PHONY: integration-test-sanitizer
integration-test-sanitizer:
@$(call make-clang, ASAN_SYMBOLIZER_PATH=/usr/lib/llvm-12/bin/llvm-symbolizer ASAN_OPTIONS=symbolize=1 \
Expand Down

0 comments on commit 6625620

Please sign in to comment.