diff options
author | Prashanth Pai <ppai@redhat.com> | 2018-05-10 18:25:56 +0530 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-05-18 04:22:42 +0000 |
commit | 89e1a4e5720509c096ff6860c1a8c483e0019278 (patch) | |
tree | 88f1fabf37eff57db85c23822f305f4f16c4e280 /xlators/experimental | |
parent | c2cf3f686f3ea0efd936d2eafc404fc9d2e0acc7 (diff) |
build: Disallow unresolved symbol references
In the past, it was often[1] forgotten for xlators to be linked against
the symbols they refer to. This often caused glusterd2 to fail while
loading xlator's shared object (.so) file.
This change adds "--no-undefined" as a linker flag which causes the
linker to treat unresolved symbol references as an error and hence fail
linking.
[1]:
https://review.gluster.org/#/c/19912/
https://review.gluster.org/#/c/19664/
https://review.gluster.org/#/c/19056/
https://review.gluster.org/#/c/17659/
https://bugzilla.redhat.com/show_bug.cgi?id=1532238
Bonus:
Added cloudsync and utime xlator's generated source files to .gitignore
Updates: bz#1193929
Change-Id: I9604a4a87b7313a5fa43bda5fdb37dfa7ef8facd
Signed-off-by: Prashanth Pai <ppai@redhat.com>
Diffstat (limited to 'xlators/experimental')
7 files changed, 7 insertions, 7 deletions
diff --git a/xlators/experimental/dht2/dht2-client/src/Makefile.am b/xlators/experimental/dht2/dht2-client/src/Makefile.am index a16b9df2f76..3a13a2a3986 100644 --- a/xlators/experimental/dht2/dht2-client/src/Makefile.am +++ b/xlators/experimental/dht2/dht2-client/src/Makefile.am @@ -6,7 +6,7 @@ dht2c_sources = dht2-client-main.c dht2common_sources = $(top_srcdir)/xlators/experimental/dht2/dht2-common/src/dht2-common-map.c dht2c_la_SOURCES = $(dht2c_sources) $(dht2common_sources) -dht2c_la_LDFLAGS = -module -avoid-version +dht2c_la_LDFLAGS = -module $(GF_XLATOR_DEFAULT_LDFLAGS) dht2c_la_LIBADD = $(top_builddir)/libglusterfs/src/libglusterfs.la AM_CFLAGS = -Wall $(GF_CFLAGS) diff --git a/xlators/experimental/dht2/dht2-server/src/Makefile.am b/xlators/experimental/dht2/dht2-server/src/Makefile.am index 12d66d126f0..c76fab0ca74 100644 --- a/xlators/experimental/dht2/dht2-server/src/Makefile.am +++ b/xlators/experimental/dht2/dht2-server/src/Makefile.am @@ -8,7 +8,7 @@ dht2s_sources = dht2-server-main.c dht2common_sources = $(top_srcdir)/xlators/experimental/dht2/dht2-common/src/dht2-common-map.c dht2s_la_SOURCES = $(dht2s_sources) $(dht2common_sources) -dht2s_la_LDFLAGS = -module -avoid-version +dht2s_la_LDFLAGS = -module $(GF_XLATOR_DEFAULT_LDFLAGS) dht2s_la_LIBADD = $(top_builddir)/libglusterfs/src/libglusterfs.la AM_CFLAGS = -Wall $(GF_CFLAGS) diff --git a/xlators/experimental/fdl/src/Makefile.am b/xlators/experimental/fdl/src/Makefile.am index da80ce28317..f39978c3930 100644 --- a/xlators/experimental/fdl/src/Makefile.am +++ b/xlators/experimental/fdl/src/Makefile.am @@ -6,7 +6,7 @@ endif noinst_HEADERS = fdl.h nodist_fdl_la_SOURCES = fdl.c -fdl_la_LDFLAGS = -module -avoid-version +fdl_la_LDFLAGS = -module $(GF_XLATOR_DEFAULT_LDFLAGS) fdl_la_LIBADD = $(top_builddir)/libglusterfs/src/libglusterfs.la if WITH_SERVER diff --git a/xlators/experimental/jbr-client/src/Makefile.am b/xlators/experimental/jbr-client/src/Makefile.am index a894e69c8d7..15616d13798 100644 --- a/xlators/experimental/jbr-client/src/Makefile.am +++ b/xlators/experimental/jbr-client/src/Makefile.am @@ -6,7 +6,7 @@ xlatordir = $(libdir)/glusterfs/$(PACKAGE_VERSION)/xlator/experimental nodist_jbrc_la_SOURCES = jbrc-cg.c CLEANFILES = $(nodist_jbrc_la_SOURCES) -jbrc_la_LDFLAGS = -module -avoid-version +jbrc_la_LDFLAGS = -module $(GF_XLATOR_DEFAULT_LDFLAGS) jbrc_la_LIBADD = $(top_builddir)/libglusterfs/src/libglusterfs.la noinst_HEADERS = $(top_srcdir)/xlators/lib/src/libxlator.h \ diff --git a/xlators/experimental/jbr-server/src/Makefile.am b/xlators/experimental/jbr-server/src/Makefile.am index fe1342dbaff..b3ceb2d9eda 100644 --- a/xlators/experimental/jbr-server/src/Makefile.am +++ b/xlators/experimental/jbr-server/src/Makefile.am @@ -6,7 +6,7 @@ xlatordir = $(libdir)/glusterfs/$(PACKAGE_VERSION)/xlator/experimental nodist_jbr_la_SOURCES = jbr-cg.c CLEANFILES = $(nodist_jbr_la_SOURCES) -jbr_la_LDFLAGS = -module -avoid-version +jbr_la_LDFLAGS = -module $(GF_XLATOR_DEFAULT_LDFLAGS) jbr_la_LIBADD = $(top_builddir)/libglusterfs/src/libglusterfs.la \ $(top_builddir)/api/src/libgfapi.la diff --git a/xlators/experimental/posix2/ds/src/Makefile.am b/xlators/experimental/posix2/ds/src/Makefile.am index d77ef8cb540..7a792a8d07b 100644 --- a/xlators/experimental/posix2/ds/src/Makefile.am +++ b/xlators/experimental/posix2/ds/src/Makefile.am @@ -6,7 +6,7 @@ xlatordir = $(libdir)/glusterfs/$(PACKAGE_VERSION)/xlator/experimental posix2_ds_sources = posix2-ds-main.c posix2_ds_la_SOURCES = $(posix2_ds_sources) -posix2_ds_la_LDFLAGS = -module -avoid-version +posix2_ds_la_LDFLAGS = -module $(GF_XLATOR_DEFAULT_LDFLAGS) posix2_ds_la_LIBADD = $(top_builddir)/libglusterfs/src/libglusterfs.la posix2_ds_la_LIBADD += $(top_builddir)/xlators/experimental/posix2/common/src/libposix2common.la diff --git a/xlators/experimental/posix2/mds/src/Makefile.am b/xlators/experimental/posix2/mds/src/Makefile.am index c6411f46114..0681cb73c45 100644 --- a/xlators/experimental/posix2/mds/src/Makefile.am +++ b/xlators/experimental/posix2/mds/src/Makefile.am @@ -6,7 +6,7 @@ xlatordir = $(libdir)/glusterfs/$(PACKAGE_VERSION)/xlator/experimental posix2_mds_sources = posix2-mds-main.c posix2_mds_la_SOURCES = $(posix2_mds_sources) -posix2_mds_la_LDFLAGS = -module -avoid-version +posix2_mds_la_LDFLAGS = -module $(GF_XLATOR_DEFAULT_LDFLAGS) posix2_mds_la_LIBADD = $(top_builddir)/libglusterfs/src/libglusterfs.la posix2_mds_la_LIBADD += $(top_builddir)/xlators/experimental/posix2/common/src/libposix2common.la |