Skip to content

Commit

Permalink
Ask libtool to stop hiding some errors
Browse files Browse the repository at this point in the history
For openzfs#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 openzfs#13086
  • Loading branch information
rincebrain authored and BrainSlayer committed Apr 11, 2022
1 parent 5a3f2fd commit 58b1c44
Show file tree
Hide file tree
Showing 15 changed files with 36 additions and 0 deletions.
2 changes: 2 additions & 0 deletions lib/libavl/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ VPATH = $(top_srcdir)/module/avl/
# Includes kernel code, generate warnings for large stack frames
AM_CFLAGS += $(FRAME_LARGER_THAN)
AM_CFLAGS += -fvisibility=hidden
# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libavl.la

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

AM_CFLAGS += $(LIBUUID_CFLAGS) $(ZLIB_CFLAGS)
AM_CFLAGS += -fvisibility=hidden
# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

noinst_LTLIBRARIES = libefi.la

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
2 changes: 2 additions & 0 deletions lib/libnvpair/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ VPATH = \
# and required CFLAGS for libtirpc
AM_CFLAGS += $(FRAME_LARGER_THAN) $(LIBTIRPC_CFLAGS)
AM_CFLAGS += -fvisibility=hidden
# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

# wchar_t is undefined-signedness, but we compare to >=0; this warns with unsigned wchar_t
libnvpair_json.$(OBJEXT): CFLAGS += -Wno-type-limits
Expand Down
2 changes: 2 additions & 0 deletions lib/libshare/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ include $(top_srcdir)/config/Rules.am
DEFAULT_INCLUDES += -I$(srcdir)

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

noinst_LTLIBRARIES = libshare.la

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
4 changes: 4 additions & 0 deletions lib/libtpool/Makefile.am
Original file line number Diff line number Diff line change
@@ -1,9 +1,13 @@
include $(top_srcdir)/config/Rules.am

AM_CFLAGS += -fvisibility=hidden

# 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
2 changes: 2 additions & 0 deletions lib/libzfs/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ VPATH = \
# Suppress unused but set variable warnings often due to ASSERTs
AM_CFLAGS += $(LIBCRYPTO_CFLAGS) $(ZLIB_CFLAGS)
AM_CFLAGS += -fvisibility=hidden
# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

pkgconfig_DATA = libzfs.pc

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

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

lib_LTLIBRARIES = libzfs_core.la

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

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

lib_LTLIBRARIES = libzfsbootenv.la

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
2 changes: 2 additions & 0 deletions lib/libzutil/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ include $(top_srcdir)/config/Rules.am

AM_CFLAGS += $(LIBBLKID_CFLAGS) $(LIBUDEV_CFLAGS)
AM_CFLAGS += -fvisibility=hidden
# See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54020
AM_CFLAGS += -no-suppress

DEFAULT_INCLUDES += -I$(srcdir)

Expand Down

0 comments on commit 58b1c44

Please sign in to comment.