Merge pull request #5842 from kmk3/build-enable-warnings

build: enable compiler warnings by default

4 files changed, 12 insertions, 8 deletions

diff --git a/configure b/configure
index dd210cd67..ceb09bd31 100755
--- a/configure
+++ b/configure
@@ -1391,7 +1391,7 @@ Optional Features:
   --disable-x11           disable X11 sandboxing support
   --disable-file-transfer disable file transfer
   --disable-suid          install as a non-SUID executable
-  --enable-fatal-warnings -W -Wall -Werror
+  --enable-fatal-warnings -W -Werror
   --enable-busybox-workaround
                           enable busybox workaround
   --enable-gcov           Gcov instrumentation
@@ -3601,7 +3601,7 @@ fi
 
 if test "x$enable_fatal_warnings" = "xyes"; then :
 
-        HAVE_FATAL_WARNINGS="-W -Wall -Werror"
+        HAVE_FATAL_WARNINGS="-W -Werror"
 
 fi
 
diff --git a/configure.ac b/configure.ac
index 357d1da45..30b031801 100644
--- a/configure.ac
+++ b/configure.ac
@@ -215,9 +215,9 @@ AS_IF([test "x$enable_suid" != "xno"], [
 HAVE_FATAL_WARNINGS=""
 AC_SUBST([HAVE_FATAL_WARNINGS])
 AC_ARG_ENABLE([fatal_warnings],
-    [AS_HELP_STRING([--enable-fatal-warnings], [-W -Wall -Werror])])
+    [AS_HELP_STRING([--enable-fatal-warnings], [-W -Werror])])
 AS_IF([test "x$enable_fatal_warnings" = "xyes"], [
-        HAVE_FATAL_WARNINGS="-W -Wall -Werror"
+        HAVE_FATAL_WARNINGS="-W -Werror"
 ])
 
 BUSYBOX_WORKAROUND="no"
diff --git a/src/prog.mk b/src/prog.mk
index b2ccf6147..e4473184f 100644
--- a/src/prog.mk
+++ b/src/prog.mk
@@ -10,8 +10,10 @@ SRCS := $(sort $(wildcard *.c)) $(MOD_SRCS)
 OBJS := $(SRCS:.c=.o) $(MOD_OBJS)
 
 PROG_CFLAGS = \
-        -ggdb $(HAVE_FATAL_WARNINGS) -O2 -DVERSION='"$(VERSION)"' \
-        -fstack-protector-all -D_FORTIFY_SOURCE=2 -Wformat -Wformat-security \
+        -ggdb -O2 -DVERSION='"$(VERSION)"' \
+        -Wall -Wextra $(HAVE_FATAL_WARNINGS) \
+        -Wformat -Wformat-security \
+        -fstack-protector-all -D_FORTIFY_SOURCE=2 \
         -fPIE \
         -DPREFIX='"$(prefix)"' -DSYSCONFDIR='"$(sysconfdir)/firejail"' \
         -DLIBDIR='"$(libdir)"' -DBINDIR='"$(bindir)"' \
diff --git a/src/so.mk b/src/so.mk
index 28bd229e5..e464a7bff 100644
--- a/src/so.mk
+++ b/src/so.mk
@@ -10,8 +10,10 @@ SRCS := $(sort $(wildcard *.c)) $(MOD_SRCS)
 OBJS := $(SRCS:.c=.o) $(MOD_OBJS)
 
 SO_CFLAGS = \
-        -ggdb $(HAVE_FATAL_WARNINGS) -O2 -DVERSION='"$(VERSION)"' \
-        -fstack-protector-all -D_FORTIFY_SOURCE=2 -Wformat -Wformat-security \
+        -ggdb -O2 -DVERSION='"$(VERSION)"' \
+        -Wall -Wextra $(HAVE_FATAL_WARNINGS) \
+        -Wformat -Wformat-security \
+        -fstack-protector-all -D_FORTIFY_SOURCE=2 \
         -fPIC
 
 SO_LDFLAGS = -pie -fPIE -Wl,-z,relro -Wl,-z,now