summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAmitay Isaacs <amitay@gmail.com>2014-06-06 16:46:15 +1000
committerMartin Schwenke <martins@samba.org>2014-06-12 05:40:10 +0200
commit71ed758c72db36234209428fa54be82fe8f0899a (patch)
tree32d9522c88cf31404c4e712e588327a3c9323a1f
parente7c72588d187f9505444f48a1c5b19e47b15ccd9 (diff)
downloadsamba-71ed758c72db36234209428fa54be82fe8f0899a.tar.gz
samba-71ed758c72db36234209428fa54be82fe8f0899a.tar.xz
samba-71ed758c72db36234209428fa54be82fe8f0899a.zip
ctdb-daemon: Remove duplicate code from helper binaries
Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net>
-rwxr-xr-xctdb/Makefile.in11
-rw-r--r--ctdb/server/ctdb_event_helper.c12
-rw-r--r--ctdb/server/ctdb_lock_helper.c2
3 files changed, 9 insertions, 16 deletions
diff --git a/ctdb/Makefile.in b/ctdb/Makefile.in
index 7222fc4e9b9..f7d45ecf194 100755
--- a/ctdb/Makefile.in
+++ b/ctdb/Makefile.in
@@ -180,13 +180,16 @@ bin/ltdbtool: tools/ltdbtool.o $(TDB_OBJ)
@echo Linking $@
$(WRAPPER) $(CC) $(CFLAGS) -o $@ $+ $(TDB_LIBS) $(LIB_FLAGS)
-bin/ctdb_lock_helper: server/ctdb_lock_helper.o lib/util/util_file.o $(CTDB_EXTERNAL_OBJ)
+HELPER_COMMON_OBJ = common/system_util.o common/ctdb_logging.o \
+ lib/util/debug.o lib/util/util_time.o lib/util/util_file.o
+
+bin/ctdb_lock_helper: server/ctdb_lock_helper.o $(HELPER_COMMON_OBJ) $(CTDB_EXTERNAL_OBJ)
@echo Linking $@
- $(WRAPPER) $(CC) $(CFLAGS) -o $@ server/ctdb_lock_helper.o lib/util/util_file.o $(CTDB_EXTERNAL_OBJ) $(TDB_LIBS) $(LIB_FLAGS)
+ $(WRAPPER) $(CC) $(CFLAGS) -o $@ server/ctdb_lock_helper.o $(HELPER_COMMON_OBJ) $(CTDB_EXTERNAL_OBJ) $(LIB_FLAGS)
-bin/ctdb_event_helper: server/ctdb_event_helper.o $(REPLACE_OBJ) $(SOCKET_WRAPPER_OBJ)
+bin/ctdb_event_helper: server/ctdb_event_helper.o $(HELPER_COMMON_OBJ) $(CTDB_EXTERNAL_OBJ)
@echo Linking $@
- $(WRAPPER) $(CC) $(CFLAGS) -o $@ server/ctdb_event_helper.o $(REPLACE_OBJ) $(SOCKET_WRAPPER_OBJ) $(LIB_FLAGS)
+ $(WRAPPER) $(CC) $(CFLAGS) -o $@ server/ctdb_event_helper.o $(HELPER_COMMON_OBJ) $(CTDB_EXTERNAL_OBJ) $(LIB_FLAGS)
bin/smnotify: utils/smnotify/gen_xdr.o utils/smnotify/gen_smnotify.o utils/smnotify/smnotify.o $(POPT_OBJ)
@echo Linking $@
diff --git a/ctdb/server/ctdb_event_helper.c b/ctdb/server/ctdb_event_helper.c
index ab1eae44091..be36b0b72a9 100644
--- a/ctdb/server/ctdb_event_helper.c
+++ b/ctdb/server/ctdb_event_helper.c
@@ -19,6 +19,7 @@
#include "includes.h"
#include "system/filesys.h"
+#include "ctdb_private.h"
static char *progname = NULL;
@@ -38,17 +39,6 @@ static void sigterm(int sig)
_exit(0);
}
-static void set_close_on_exec(int fd)
-{
- int v;
-
- v = fcntl(fd, F_GETFD, 0);
- if (v == -1) {
- return;
- }
- fcntl(fd, F_SETFD, v | FD_CLOEXEC);
-}
-
static int check_executable(const char *path)
{
struct stat st;
diff --git a/ctdb/server/ctdb_lock_helper.c b/ctdb/server/ctdb_lock_helper.c
index d8a1d249b75..b1515a256e1 100644
--- a/ctdb/server/ctdb_lock_helper.c
+++ b/ctdb/server/ctdb_lock_helper.c
@@ -20,7 +20,7 @@
#include "includes.h"
#include "tdb.h"
#include "system/filesys.h"
-#include "../include/ctdb_private.h"
+#include "ctdb_private.h"
static char *progname = NULL;