diff options
author | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-05-31 05:12:14 -0300 |
---|---|---|
committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-05-31 05:38:02 -0300 |
commit | 94090654695bce5d00ef5a833fbd21f0d3ef195b (patch) | |
tree | b36bc36d641e5b3d6b945219ceac0d7dc0d43cd9 | |
parent | build: organize warning flags in CFLAGS (diff) | |
download | firejail-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-x | configure | 4 | ||||
-rw-r--r-- | configure.ac | 4 | ||||
-rw-r--r-- | src/prog.mk | 2 | ||||
-rw-r--r-- | src/so.mk | 2 |
4 files changed, 6 insertions, 6 deletions
@@ -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 | ||
3602 | if test "x$enable_fatal_warnings" = "xyes"; then : | 3602 | if test "x$enable_fatal_warnings" = "xyes"; then : |
3603 | 3603 | ||
3604 | HAVE_FATAL_WARNINGS="-W -Wall -Werror" | 3604 | HAVE_FATAL_WARNINGS="-W -Werror" |
3605 | 3605 | ||
3606 | fi | 3606 | fi |
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"], [ | |||
215 | HAVE_FATAL_WARNINGS="" | 215 | HAVE_FATAL_WARNINGS="" |
216 | AC_SUBST([HAVE_FATAL_WARNINGS]) | 216 | AC_SUBST([HAVE_FATAL_WARNINGS]) |
217 | AC_ARG_ENABLE([fatal_warnings], | 217 | AC_ARG_ENABLE([fatal_warnings], |
218 | [AS_HELP_STRING([--enable-fatal-warnings], [-W -Wall -Werror])]) | 218 | [AS_HELP_STRING([--enable-fatal-warnings], [-W -Werror])]) |
219 | AS_IF([test "x$enable_fatal_warnings" = "xyes"], [ | 219 | AS_IF([test "x$enable_fatal_warnings" = "xyes"], [ |
220 | HAVE_FATAL_WARNINGS="-W -Wall -Werror" | 220 | HAVE_FATAL_WARNINGS="-W -Werror" |
221 | ]) | 221 | ]) |
222 | 222 | ||
223 | BUSYBOX_WORKAROUND="no" | 223 | BUSYBOX_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 | ||
12 | PROG_CFLAGS = \ | 12 | PROG_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 \ |
@@ -11,7 +11,7 @@ OBJS := $(SRCS:.c=.o) $(MOD_OBJS) | |||
11 | 11 | ||
12 | SO_CFLAGS = \ | 12 | SO_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 |