scripts/build_docker: abort package builds on compiler warnings

Don't allow any more compile warnings to creep in by aborting package
builds when compiler warnings are encountered.

The restriction is relaxed for test runs so devs may still use CI to
test work in progress.

Author: zultron

.travis.yml

@@ -48,7 +48,7 @@ services:
   - docker
 
 before_install:
-  - .travis/check_sftp.sh 
+  - .travis/check_sftp.sh
 
 env:
   global:

scripts/build_docker

@@ -109,6 +109,7 @@ case $CMD in
 	    -e TRAVIS_PULL_REQUEST=$TRAVIS_PULL_REQUEST
 	    -e TRAVIS_REPO=$TRAVIS_REPO
 	    -e TRAVIS_REPO_SLUG=$TRAVIS_REPO_SLUG
+            -e DPKG_CFLAGS="${DPKG_CFLAGS} -Werror"
 	)
 
 	BUILD_CL=( bash -xec "

src/Makefile

@@ -371,7 +371,8 @@ DEBUG := -g -Wall -funwind-tables
 PROFILE_CFLAGS :=
 PROFILE_LDFLAGS :=
 
-CFLAGS := $(INCLUDE) $(OPT) $(DEBUG) $(PROFILE_CFLAGS) -DULAPI $(call cc-option,-std=gnu99 -fgnu89-inline)
+CFLAGS := $(INCLUDE) $(OPT) $(DEBUG) $(DPKG_CFLAGS) $(PROFILE_CFLAGS) \
+	-DULAPI $(call cc-option,-std=gnu99 -fgnu89-inline)
 
 # enable runtime comparison of GIT_CONFIG_SHA (config.h) and GIT_BUILD_SHA
 CFLAGS += -DGIT_BUILD_SHA=\"'$(GIT_BUILD_SHA)'\"
@@ -392,7 +393,7 @@ BASE_CXXFLAGS:= $(INCLUDE) $(CXXFLAGS) \
 		$(ARCH_CFLAGS)
 
 CXXFLAGS :=   $(BASE_CXXFLAGS) -DULAPI
-RT_CXXFLAGS :=   $(BASE_CXXFLAGS) 
+RT_CXXFLAGS :=   $(BASE_CXXFLAGS)
 
 ifeq ($(RUN_IN_PLACE),yes)
 LDFLAGS += -L$(LIB_DIR) -Wl,-rpath,$(LIB_DIR)
@@ -1672,7 +1673,8 @@ $(sort $(RTOBJS)) : $(OBJDIR)/%.o : %.c $(DEPDIR)/%.d
 	$(ECHO) Compiling realtime $(threads) $<
 	@rm -f $@
 	@mkdir -p $(dir $@)
-	$(Q)$(CC) -c $(OPT) $(DEBUG) $(EXTRA_CFLAGS) $< -o $@
+	$(Q)$(CC) -c $(OPT) $(DPKG_CFLAGS) $(DEBUG) \
+	    $(EXTRA_CFLAGS) $< -o $@
 
 # Rules to make .o (object) files from .cc
 $(sort $(CXXRTOBJS)) : $(OBJDIR)/%.o : %.cc $(DEPDIR)/%.d