CI: warnings as errors for safety

dzid26 · Apr 10, 2024 · d4a8e02 · d4a8e02
1 parent 80333a0
commit d4a8e02
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -29,7 +29,8 @@ jobs:
     - name: Build
       run: |
         cd src/controller
-        ./compile.bat
+        ../../tools/cygwin/bin/make.exe clean
+        ../../tools/cygwin/bin/make.exe CFLAGS=--Werror
     - uses: actions/upload-artifact@v3
       with:
         name: firmware
@@ -53,7 +54,7 @@ jobs:
         run: |
           cd src/controller
           make clean
-          make
+          make CFLAGS=--Werror
       - uses: actions/upload-artifact@v3
         with:
           name: firmware

diff --git a/src/controller/Makefile b/src/controller/Makefile
@@ -65,9 +65,9 @@ ebike_app.h pins.h eeprom.h lights.h config.h
 # The list of .rel files can be derived from the list of their source files
 RELS = $(addprefix $(ODIR)/,$(notdir $(EXTRASRCS:.c=.rel)))
 
-INCLUDES = -I$(IDIR) -I$(IDIR1)
-CFLAGS = -m$(PLATFORM) -D$(DEVICE) -Ddouble=float --std-c99 --nolospre --opt-code-speed --peep-asm --peep-file peep.txt
-DEBUG_FLAGS = --out-fmt-ihx --debug
+override INCLUDES += -I$(IDIR) -I$(IDIR1)
+override CFLAGS += -m$(PLATFORM) -D$(DEVICE) -Ddouble=float --std-c99 --nolospre --opt-code-speed --peep-asm --peep-file peep.txt
+override DEBUG_FLAGS += --out-fmt-ihx --debug
 LIBS = 
 
 # This just provides the conventional target name "all"; it is optional