Skip to content

Commit

Permalink
Ask libtool to stop hiding some errors
Browse files Browse the repository at this point in the history
For #13083, curiously, it did not print the actual error, just
that the compile failed with "Error 1".

In theory, this flag should cause it to report errors twice sometimes.
In practice, I'm pretty okay with reporting some twice if it avoids
reporting some never.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Damian Szuberski <szuberskidamian@gmail.com>
Signed-off-by: Rich Ercolani <rincebrain@gmail.com>
Closes #13086
  • Loading branch information
rincebrain authored and tonyhutter committed Sep 21, 2022
1 parent d05f303 commit ebbbe01
Show file tree
Hide file tree
Showing 15 changed files with 43 additions and 0 deletions.
3 changes: 3 additions & 0 deletions lib/libavl/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@ VPATH = $(top_srcdir)/module/avl/
# Includes kernel code, generate warnings for large stack frames
AM_CFLAGS += $(FRAME_LARGER_THAN)

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libavl.la

KERNEL_C = \
Expand Down
3 changes: 3 additions & 0 deletions lib/libefi/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ include $(top_srcdir)/config/Rules.am

AM_CFLAGS += $(LIBUUID_CFLAGS) $(ZLIB_CFLAGS)

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libefi.la

USER_C = \
Expand Down
2 changes: 2 additions & 0 deletions lib/libicp/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ VPATH = \

# Includes kernel code, generate warnings for large stack frames
AM_CFLAGS += $(FRAME_LARGER_THAN)
# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libicp.la

Expand Down
3 changes: 3 additions & 0 deletions lib/libnvpair/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ VPATH = \
# and required CFLAGS for libtirpc
AM_CFLAGS += $(FRAME_LARGER_THAN) $(LIBTIRPC_CFLAGS)

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

lib_LTLIBRARIES = libnvpair.la

include $(top_srcdir)/config/Abigail.am
Expand Down
3 changes: 3 additions & 0 deletions lib/libshare/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ include $(top_srcdir)/config/Rules.am

DEFAULT_INCLUDES += -I$(srcdir)

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libshare.la

USER_C = \
Expand Down
3 changes: 3 additions & 0 deletions lib/libspl/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ include $(top_srcdir)/config/Rules.am

SUBDIRS = include

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libspl_assert.la libspl.la

libspl_assert_la_SOURCES = \
Expand Down
3 changes: 3 additions & 0 deletions lib/libtpool/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,9 @@ include $(top_srcdir)/config/Rules.am
# https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61118
AM_CFLAGS += $(NO_CLOBBERED)

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libtpool.la

USER_C = \
Expand Down
3 changes: 3 additions & 0 deletions lib/libunicode/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@ VPATH = $(top_srcdir)/module/unicode
# Includes kernel code, generate warnings for large stack frames
AM_CFLAGS += $(FRAME_LARGER_THAN)

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libunicode.la

KERNEL_C = \
Expand Down
3 changes: 3 additions & 0 deletions lib/libuutil/Makefile.am
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
include $(top_srcdir)/config/Rules.am

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

lib_LTLIBRARIES = libuutil.la

include $(top_srcdir)/config/Abigail.am
Expand Down
3 changes: 3 additions & 0 deletions lib/libzfs/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ VPATH = \
# Suppress unused but set variable warnings often due to ASSERTs
AM_CFLAGS += $(LIBCRYPTO_CFLAGS) $(ZLIB_CFLAGS)

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

pkgconfig_DATA = libzfs.pc

lib_LTLIBRARIES = libzfs.la
Expand Down
3 changes: 3 additions & 0 deletions lib/libzfs_core/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ include $(top_srcdir)/config/Rules.am

pkgconfig_DATA = libzfs_core.pc

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

lib_LTLIBRARIES = libzfs_core.la

include $(top_srcdir)/config/Abigail.am
Expand Down
3 changes: 3 additions & 0 deletions lib/libzfsbootenv/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ include $(top_srcdir)/config/Rules.am

pkgconfig_DATA = libzfsbootenv.pc

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

lib_LTLIBRARIES = libzfsbootenv.la

include $(top_srcdir)/config/Abigail.am
Expand Down
3 changes: 3 additions & 0 deletions lib/libzpool/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,9 @@ AM_CFLAGS += $(ZLIB_CFLAGS)

AM_CFLAGS += -DLIB_ZPOOL_BUILD

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

lib_LTLIBRARIES = libzpool.la

USER_C = \
Expand Down
2 changes: 2 additions & 0 deletions lib/libzstd/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ VPATH = $(top_srcdir)/module/zstd
# -fno-tree-vectorize is set for gcc in zstd/common/compiler.h
# Set it for other compilers, too.
AM_CFLAGS += -fno-tree-vectorize
# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libzstd.la

Expand Down
3 changes: 3 additions & 0 deletions lib/libzutil/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ include $(top_srcdir)/config/Rules.am

AM_CFLAGS += $(LIBBLKID_CFLAGS) $(LIBUDEV_CFLAGS)

# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

DEFAULT_INCLUDES += -I$(srcdir)

noinst_LTLIBRARIES = libzutil.la
Expand Down

0 comments on commit ebbbe01

Please sign in to comment.