diff options
author | Shehjar Tikoo <shehjart@gluster.com> | 2009-06-29 09:38:29 +0000 |
---|---|---|
committer | Anand V. Avati <avati@dev.gluster.com> | 2009-06-29 10:51:16 -0700 |
commit | a3ece0caa52ad2eacf8a8691aaca53295cde972f (patch) | |
tree | 8bcb7740cc75f36fb499ac1febb3cfa8e2f1c817 /booster/src/Makefile.am | |
parent | 304e4274ca9b0339539581c5413e3339078c1182 (diff) |
booster: Fix build problems with various libtool versions
We've been facing a problem on some test systems where the
booster.so is built by libtool as an executable rather than
a dynamically loadable library.
This problem is probably caused by it seeing a _init function in
the source. This is the name of the libc initiliazation
function so it could be the source of the problem.
In any case, ld-preloadable libraries must not have a function
called _init, instead they need to have __attribute ((constructor))
as the attribute for any and all functions the library wants
executed before the program's main(). Our earlier approach was
inherently problematic.
This commit also cleans up the booster Makefile for better
build behaviour. Credit: Harsha.
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
Diffstat (limited to 'booster/src/Makefile.am')
-rw-r--r-- | booster/src/Makefile.am | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/booster/src/Makefile.am b/booster/src/Makefile.am index a51585912..6402c8d79 100644 --- a/booster/src/Makefile.am +++ b/booster/src/Makefile.am @@ -1,18 +1,21 @@ -xlatordir = $(libdir)/glusterfs/$(PACKAGE_VERSION)/xlator/performance - -ldpreload_PROGRAMS = glusterfs-booster.so +ldpreload_LTLIBRARIES = libglusterfs-booster.la ldpreloaddir = $(libdir)/glusterfs/ noinst_HEADERS = booster_fstab.h booster-fd.h -glusterfs_booster_so_SOURCES = booster.c booster_stat.c booster_fstab.c booster-fd.c -glusterfs_booster_so_CFLAGS = -I$(top_srcdir)/libglusterfsclient/src/ -D_GNU_SOURCE -D$(GF_HOST_OS) -fPIC -Wall \ - -pthread $(GF_BOOSTER_CFLAGS) -glusterfs_booster_so_CPPFLAGS = -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE \ +libglusterfs_booster_la_SOURCES = booster.c booster_stat.c booster_fstab.c booster-fd.c +libglusterfs_booster_la_CFLAGS = -I$(top_srcdir)/libglusterfsclient/src/ -D_GNU_SOURCE -D$(GF_HOST_OS) -fPIC -Wall \ + -pthread $(GF_BOOSTER_CFLAGS) -shared -nostartfiles +libglusterfs_booster_la_CPPFLAGS = -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE \ -I$(top_srcdir)/libglusterfsclient/src \ -I$(top_srcdir)/libglusterfs/src -DDATADIR=\"$(localstatedir)\" \ -DCONFDIR=\"$(sysconfdir)/glusterfs\" -glusterfs_booster_so_LDFLAGS = -shared -nostartfiles -glusterfs_booster_so_LDADD = -L$(top_builddir)/libglusterfs/src -lglusterfs \ - -L$(top_builddir)/libglusterfsclient/src -lglusterfsclient -CLEANFILES = +libglusterfs_booster_la_LDFLAGS = -module -avoidversion +libglusterfs_booster_la_LIBADD = $(top_builddir)/libglusterfs/src/libglusterfs.la $(top_builddir)/libglusterfsclient/src/libglusterfsclient.la + +CLEANFILES = + +uninstall-local: + rm -f $(DESTDIR)$(ldpreloaddir)/glusterfs-booster.so +install-data-hook: + ln -sf libglusterfs-booster.so $(DESTDIR)$(ldpreloaddir)/glusterfs-booster.so |