Removed build strict test and enabled -Werror flag

commaai · Jul 7, 2019 · ba68569 · ba68569
1 parent da11f0f
commit ba68569
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 47 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -40,19 +40,6 @@ jobs:
           path: /tmp/misra/misra_safety_output.txt
 
 
-  strict-compiler:
-    machine:
-      docker_layer_caching: true
-    steps:
-      - checkout
-      - run:
-          name: Build image
-          command: "docker build -t panda_strict_compiler -f tests/build_strict/Dockerfile ."
-      - run:
-          name: Build Panda with strict compiler rules
-          command: |
-            docker run panda_strict_compiler /bin/bash -c "cd /panda/tests/build_strict; ./test_build_strict.sh"
-
   build:
     machine:
       docker_layer_caching: true
@@ -105,6 +92,5 @@ workflows:
     jobs:
       - safety
       - misra-c2012
-      - strict-compiler
       - build
       - safety_replay
diff --git a/board/Makefile b/board/Makefile
@@ -1,5 +1,5 @@
 PROJ_NAME = panda
-CFLAGS = -g -Wall
+CFLAGS = -g -Wall -Wextra -Wstrict-prototypes -Werror
 
 CFLAGS += -mlittle-endian -mthumb -mcpu=cortex-m4
 CFLAGS += -mhard-float -DSTM32F4 -DSTM32F413xx -mfpu=fpv4-sp-d16 -fsingle-precision-constant

diff --git a/board/Makefile.strict b/board/Makefile.strict
diff --git a/tests/build_strict/Dockerfile b/tests/build_strict/Dockerfile
diff --git a/tests/build_strict/test_build_strict.sh b/tests/build_strict/test_build_strict.sh