CI: warnings as errors for safety

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

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -25,11 +25,13 @@ jobs:
       run: | #install silent and then unpack missing installation libraries
         Invoke-WebRequest https://altushost-swe.dl.sourceforge.net/project/sdcc/sdcc-win64/$env:SDCC_VERSION/sdcc-$env:SDCC_VERSION-rc3-x64-setup.exe -OutFile sdcc_setup.exe
         Start-Process -wait -FilePath "sdcc_setup.exe" -ArgumentList "/S", "/D=C:\Program Files\SDCC"
+        $env:Path += ";C:\Program Files\SDCC\bin"
     - uses: actions/checkout@v2
     - 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 +55,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