aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJack O'Connor <[email protected]>2023-01-19 13:12:21 -0800
committerJack O'Connor <[email protected]>2023-01-19 13:59:41 -0800
commit1aceb945e0d2faf9fbb3a9f89a7e4b7fca71f099 (patch)
treee56f0124acd93420981fe010d2ff8ba706e685da
parent606a5825d910249094d8d9c78f83d1cabaf5dfb2 (diff)
set CFLAGS in CI, including -Werrorwerror
These only affect the Makefile tests (which only run on Linux) since -Werror currently fails on Windows and macOS.
-rw-r--r--.github/workflows/ci.yml2
-rw-r--r--c/Makefile.testing4
2 files changed, 4 insertions, 2 deletions
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 399c042..f832263 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -178,6 +178,8 @@ jobs:
c_tests:
name: C Makefile tests
runs-on: ubuntu-latest
+ env:
+ CFLAGS: "-Wall -Wextra -pedantic -Werror -Wsign-conversion"
steps:
- uses: actions/checkout@v3
diff --git a/c/Makefile.testing b/c/Makefile.testing
index b540528..dba5da7 100644
--- a/c/Makefile.testing
+++ b/c/Makefile.testing
@@ -3,8 +3,8 @@
NAME=blake3
CC=gcc
-CFLAGS=-O3 -Wall -Wextra -std=c11 -pedantic -fstack-protector-strong -D_FORTIFY_SOURCE=2 -fPIE -fvisibility=hidden
-LDFLAGS=-pie -Wl,-z,relro,-z,now
+CFLAGS+=-O3 -Wall -Wextra -std=c11 -pedantic -fstack-protector-strong -D_FORTIFY_SOURCE=2 -fPIE -fvisibility=hidden
+LDFLAGS+=-pie -Wl,-z,relro,-z,now
TARGETS=
ASM_TARGETS=
EXTRAFLAGS=-Wa,--noexecstack