aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLibravatar Kelvin M. Klann <kmk3.code@protonmail.com>2023-05-31 05:12:14 -0300
committerLibravatar Kelvin M. Klann <kmk3.code@protonmail.com>2023-05-31 05:38:02 -0300
commit94090654695bce5d00ef5a833fbd21f0d3ef195b (patch)
treeb36bc36d641e5b3d6b945219ceac0d7dc0d43cd9
parentbuild: organize warning flags in CFLAGS (diff)
downloadfirejail-94090654695bce5d00ef5a833fbd21f0d3ef195b.tar.gz
firejail-94090654695bce5d00ef5a833fbd21f0d3ef195b.tar.zst
firejail-94090654695bce5d00ef5a833fbd21f0d3ef195b.zip
build: enable compiler warnings by default
Enable -Wall by default and add -Wextra.
-rwxr-xr-xconfigure4
-rw-r--r--configure.ac4
-rw-r--r--src/prog.mk2
-rw-r--r--src/so.mk2
4 files changed, 6 insertions, 6 deletions
diff --git a/configure b/configure
index dd210cd67..ceb09bd31 100755
--- a/configure
+++ b/configure
@@ -1391,7 +1391,7 @@ Optional Features:
1391 --disable-x11 disable X11 sandboxing support 1391 --disable-x11 disable X11 sandboxing support
1392 --disable-file-transfer disable file transfer 1392 --disable-file-transfer disable file transfer
1393 --disable-suid install as a non-SUID executable 1393 --disable-suid install as a non-SUID executable
1394 --enable-fatal-warnings -W -Wall -Werror 1394 --enable-fatal-warnings -W -Werror
1395 --enable-busybox-workaround 1395 --enable-busybox-workaround
1396 enable busybox workaround 1396 enable busybox workaround
1397 --enable-gcov Gcov instrumentation 1397 --enable-gcov Gcov instrumentation
@@ -3601,7 +3601,7 @@ fi
3601 3601
3602if test "x$enable_fatal_warnings" = "xyes"; then : 3602if test "x$enable_fatal_warnings" = "xyes"; then :
3603 3603
3604 HAVE_FATAL_WARNINGS="-W -Wall -Werror" 3604 HAVE_FATAL_WARNINGS="-W -Werror"
3605 3605
3606fi 3606fi
3607 3607
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"], [
215HAVE_FATAL_WARNINGS="" 215HAVE_FATAL_WARNINGS=""
216AC_SUBST([HAVE_FATAL_WARNINGS]) 216AC_SUBST([HAVE_FATAL_WARNINGS])
217AC_ARG_ENABLE([fatal_warnings], 217AC_ARG_ENABLE([fatal_warnings],
218 [AS_HELP_STRING([--enable-fatal-warnings], [-W -Wall -Werror])]) 218 [AS_HELP_STRING([--enable-fatal-warnings], [-W -Werror])])
219AS_IF([test "x$enable_fatal_warnings" = "xyes"], [ 219AS_IF([test "x$enable_fatal_warnings" = "xyes"], [
220 HAVE_FATAL_WARNINGS="-W -Wall -Werror" 220 HAVE_FATAL_WARNINGS="-W -Werror"
221]) 221])
222 222
223BUSYBOX_WORKAROUND="no" 223BUSYBOX_WORKAROUND="no"
diff --git a/src/prog.mk b/src/prog.mk
index 537244656..e4473184f 100644
--- a/src/prog.mk
+++ b/src/prog.mk
@@ -11,7 +11,7 @@ OBJS := $(SRCS:.c=.o) $(MOD_OBJS)
11 11
12PROG_CFLAGS = \ 12PROG_CFLAGS = \
13 -ggdb -O2 -DVERSION='"$(VERSION)"' \ 13 -ggdb -O2 -DVERSION='"$(VERSION)"' \
14 $(HAVE_FATAL_WARNINGS) \ 14 -Wall -Wextra $(HAVE_FATAL_WARNINGS) \
15 -Wformat -Wformat-security \ 15 -Wformat -Wformat-security \
16 -fstack-protector-all -D_FORTIFY_SOURCE=2 \ 16 -fstack-protector-all -D_FORTIFY_SOURCE=2 \
17 -fPIE \ 17 -fPIE \
diff --git a/src/so.mk b/src/so.mk
index 8787c71b8..e464a7bff 100644
--- a/src/so.mk
+++ b/src/so.mk
@@ -11,7 +11,7 @@ OBJS := $(SRCS:.c=.o) $(MOD_OBJS)
11 11
12SO_CFLAGS = \ 12SO_CFLAGS = \
13 -ggdb -O2 -DVERSION='"$(VERSION)"' \ 13 -ggdb -O2 -DVERSION='"$(VERSION)"' \
14 $(HAVE_FATAL_WARNINGS) \ 14 -Wall -Wextra $(HAVE_FATAL_WARNINGS) \
15 -Wformat -Wformat-security \ 15 -Wformat -Wformat-security \
16 -fstack-protector-all -D_FORTIFY_SOURCE=2 \ 16 -fstack-protector-all -D_FORTIFY_SOURCE=2 \
17 -fPIC 17 -fPIC