changeset 11261:f0359b41944f

Make it easier to use the 'visibility' module.
author Bruno Haible <bruno@clisp.org>
date Wed, 04 Mar 2009 12:04:32 +0100
parents 78ac16260082
children 06ebfb5a6274
files ChangeLog gnulib-tool modules/visibility
diffstat 3 files changed, 13 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2009-03-04  Bruno Haible  <bruno@clisp.org>
+
+	* modules/visibility (Makefile.am): Augment AM_CFLAGS.
+	* gnulib-tool (func_emit_lib_Makefile_am): Emit initialization of
+	AM_CFLAGS.
+	Reported by Simon Josefsson.
+
 2009-03-03  Bruno Haible  <bruno@clisp.org>
 
 	* doc/visibility.texi: Recommend to use HAVE_VISIBILITY as a C macro.
--- a/gnulib-tool
+++ b/gnulib-tool
@@ -1932,6 +1932,7 @@
   if test -z "$makefile_name"; then
     echo
     echo "AM_CPPFLAGS ="
+    echo "AM_CFLAGS ="
   fi
   echo
   if LC_ALL=C grep "^[a-zA-Z0-9_]*_${perhapsLT}LIBRARIES *+\{0,1\}= *$libname\\.$libext\$" allsnippets.tmp > /dev/null; then
--- a/modules/visibility
+++ b/modules/visibility
@@ -10,6 +10,11 @@
 gl_VISIBILITY
 
 Makefile.am:
+# The value of $(CFLAG_VISIBILITY) needs to be added to the CFLAGS for the
+# compilation of all sources that make up the library. This line here does it
+# only for the gnulib part of it. The developer is responsible for adding
+# $(CFLAG_VISIBILITY) to the Makefile.ams of the other portions of the library.
+AM_CFLAGS += $(CFLAG_VISIBILITY)
 
 Include: