Browse Source

Merged in the ideas from libmemcached in terms of hiding internal symbols.

Monty Taylor 15 years ago
parent
commit
a67757f490
4 changed files with 7 additions and 6 deletions
  1. 2 1
      libgearman-server/include.am
  2. 1 1
      libgearman/include.am
  3. 2 2
      libgearman/visibility.h
  4. 2 2
      tests/include.am

+ 2 - 1
libgearman-server/include.am

@@ -93,4 +93,5 @@ libgearman_server_libgearman_server_la_LIBADD= \
 	$(LTLIBMEMCACHED) \
 	$(LTLIBPQ) \
 	$(LTLIBSQLITE3) \
-	libgearman/libgearman.la
+	libgearman/libgearman.la \
+	libgearman/libgearmancore.la

+ 1 - 1
libgearman/include.am

@@ -28,7 +28,7 @@ noinst_HEADERS+= \
 		 libgearman/common.h \
 		 libgearman/log.h
 
-lib_LTLIBRARIES+= libgearman/libgearmancore.la
+noinst_LTLIBRARIES+= libgearman/libgearmancore.la
 libgearman_libgearmancore_la_SOURCES= \
 				       libgearman/connection.c \
 				       libgearman/core.c \

+ 2 - 2
libgearman/visibility.h

@@ -26,12 +26,12 @@
 #if defined(BUILDING_LIBGEARMAN)
 # if defined(HAVE_VISIBILITY)
 #  define GEARMAN_API __attribute__ ((visibility("default")))
-#  define GEARMAN_INTERNAL_API __attribute__ ((visibility("default")))
+#  define GEARMAN_INTERNAL_API __attribute__ ((visibility("hidden")))
 #  define GEARMAN_API_DEPRECATED __attribute__ ((deprecated,visibility("default")))
 #  define GEARMAN_LOCAL  __attribute__ ((visibility("hidden")))
 # elif defined (__SUNPRO_C) && (__SUNPRO_C >= 0x550)
 #  define GEARMAN_API __global
-#  define GEARMAN_INTERNAL_API __global
+#  define GEARMAN_INTERNAL_API __hidden
 #  define GEARMAN_API_DEPRECATED __global
 #  define GEARMAN_LOCAL __hidden
 # elif defined(_MSC_VER)

+ 2 - 2
tests/include.am

@@ -54,10 +54,10 @@ tests_client_test_SOURCES= tests/test_worker.c tests/client_test.c
 tests_client_test_LDADD= ${TEST_LDADD}
 
 tests_internals_test_SOURCES= tests/internals.c
-tests_internals_test_LDADD= ${TEST_LDADD}
+tests_internals_test_LDADD= ${TEST_LDADD} libgearman/libgearmancore.la
 
 tests_regression_test_SOURCES= tests/regression.c
-tests_regression_test_LDADD= ${TEST_LDADD}
+tests_regression_test_LDADD= ${TEST_LDADD} libgearman/libgearmancore.la
 
 tests_worker_test_SOURCES= tests/worker_test.c
 tests_worker_test_LDADD= ${TEST_LDADD}