piotr/edgetx-gx12
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Re #2908: compiler warnings treated as errors (use build option WARNINGS_AS_ERRORS=YES), enabled in commit tests for several platforms (where it passes)

Browse Source
This commit is contained in:
Damjan Adamic
2015-09-20 13:38:35 +02:00
parent bfc01f9504
commit 701af4984b
4 changed files with 30 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
radio/src/Makefile
+9 -1
View File
@@ -375,6 +375,10 @@ SIMU_AUDIO = NO
# Values = NO, YES
WATCHDOG_TEST = NO
# Treat compiler warnings as errors
# Values = NO, YES
WARNINGS_AS_ERRORS = NO
#------- END BUILD OPTIONS ---------------------------
# Define programs and commands.
@@ -1427,6 +1431,10 @@ CPPFLAGS += -Wno-strict-aliasing
CPPFLAGS += -Wformat -Wreturn-type -Wunused -Wuninitialized -Wunknown-pragmas -Wno-switch -Wtype-limits
INCFLAGS = $(patsubst %,-I%,$(INCDIRS))
ifeq ($(WARNINGS_AS_ERRORS), YES)
CPPFLAGS += -Werror
endif
MATH_LIB = -lm
#---------------- Linker Options ----------------
@@ -1493,7 +1501,7 @@ bootloader.lbm: targets/taranis/bootloader/bootloader_ramBoot.bin
$(BIN2LBM) $^ $@
.PHONY targets/taranis/bootloader/bootloader_ramBoot.bin:
@echo "Building bootloader..."
$(MAKE) -C targets/taranis/bootloader PCB=$(PCB) PCBREV=$(PCBREV) NANO=$(NANO)
$(MAKE) -C targets/taranis/bootloader PCB=$(PCB) PCBREV=$(PCBREV) NANO=$(NANO) WARNINGS_AS_ERRORS=$(WARNINGS_AS_ERRORS)
else ifeq ($(PCB), $(filter $(PCB), 9XRPRO AR9X))
bootloader.lbm:
@echo "Fetching Mike's bootloader..."
radio/src/targets/taranis/bootloader/Makefile
+5
View File
@@ -240,6 +240,11 @@ CPFLAGS = $(MCFLAGS) $(OPT) -gdwarf-2 -mthumb -fomit-frame-pointer -Wall -Wstric
LDFLAGS = $(MCFLAGS) -mthumb -nostartfiles -T$(LDSCRIPT) -Wl,-Map=$(FULL_PRJ).map,--cref,--no-warn-mismatch,--gc-sections $(LIBDIR) $(NEWLIB_NANO_FLAGS)
CPPFLAGS = $(MCFLAGS) $(OPT) -gdwarf-2 -mthumb -fomit-frame-pointer -fdata-sections -ffunction-sections -Wall -fverbose-asm $(DEFS)
ifeq ($(WARNINGS_AS_ERRORS), YES)
CPFLAGS += -Werror
CPPFLAGS += -Werror
endif
CPPFLAGS += $(CPPDEFS)
CPFLAGS += $(CPPDEFS)
CPFLAGS += $(patsubst %,-I%,$(EXTRAINCDIRS))
radio/src/targets/taranis/usb_driver.c
+1
View File
@@ -35,6 +35,7 @@
*/
#include "board_taranis.h"
#include "STM32_USB-Host-Device_Lib_V2.1.0/Libraries/STM32_USB_OTG_Driver/inc/usb_dcd_int.h"
int usbPlugged(void)
{
radio/util/commit-tests.sh
+15 -15
View File
@@ -8,8 +8,8 @@ cd radio/src/
# OpenTX on 9X stock
make clean
make PCB=9X EXT=FRSKY HELI=YES
make simu PCB=9X EXT=FRSKY HELI=YES
make PCB=9X EXT=FRSKY HELI=YES
make simu PCB=9X EXT=FRSKY HELI=YES
make gtests EXT=FRSKY HELI=YES
./gtests
@@ -19,35 +19,35 @@ make PCB=9X EXT=MAVLINK HELI=YES
# OpenTX on Taranis X9E
make clean
make PCB=TARANIS PCBREV=REV9E HELI=YES GVARS=YES LUA=YES
make simu PCB=TARANIS PCBREV=REV9E HELI=YES GVARS=YES LUA=YES
make gtests PCB=TARANIS PCBREV=REV9E HELI=YES GVARS=YES LUA=YES
make PCB=TARANIS PCBREV=REV9E HELI=YES GVARS=YES LUA=YES WARNINGS_AS_ERRORS=YES
make simu PCB=TARANIS PCBREV=REV9E HELI=YES GVARS=YES LUA=YES WARNINGS_AS_ERRORS=YES
make gtests PCB=TARANIS PCBREV=REV9E HELI=YES GVARS=YES LUA=YES WARNINGS_AS_ERRORS=YES
./gtests
# OpenTX on Taranis Plus
make clean
make PCB=TARANIS PCBREV=REVPLUS HELI=YES LUA=YES
make simu PCB=TARANIS PCBREV=REVPLUS HELI=YES LUA=YES
make gtests PCB=TARANIS PCBREV=REVPLUS HELI=YES LUA=YES
make PCB=TARANIS PCBREV=REVPLUS HELI=YES LUA=YES WARNINGS_AS_ERRORS=YES
make simu PCB=TARANIS PCBREV=REVPLUS HELI=YES LUA=YES WARNINGS_AS_ERRORS=YES
make gtests PCB=TARANIS PCBREV=REVPLUS HELI=YES LUA=YES WARNINGS_AS_ERRORS=YES
./gtests
# OpenTX on Taranis
make clean
make PCB=TARANIS HELI=YES LUA=YES
make simu PCB=TARANIS HELI=YES LUA=YES
make gtests PCB=TARANIS HELI=YES LUA=YES
make PCB=TARANIS HELI=YES LUA=YES WARNINGS_AS_ERRORS=YES
make simu PCB=TARANIS HELI=YES LUA=YES WARNINGS_AS_ERRORS=YES
make gtests PCB=TARANIS HELI=YES LUA=YES WARNINGS_AS_ERRORS=YES
./gtests
# OpenTX on Sky9x
make clean
make PCB=SKY9X HELI=YES
make simu PCB=SKY9X HELI=YES
make gtests PCB=SKY9X HELI=YES
make PCB=SKY9X HELI=YES WARNINGS_AS_ERRORS=YES
make simu PCB=SKY9X HELI=YES WARNINGS_AS_ERRORS=YES
make gtests PCB=SKY9X HELI=YES WARNINGS_AS_ERRORS=YES
./gtests
# OpenTX on Gruvin9x
make clean
make PCB=GRUVIN9X EXT=FRSKY SDCARD=YES
make PCB=GRUVIN9X EXT=FRSKY SDCARD=YES
make simu PCB=GRUVIN9X EXT=FRSKY SDCARD=YES
# OpenTX on MEGA2560