visibility: Use a separate CFLAGS variable
authorRyan Lortie <desrt@desrt.ca>
Tue, 15 Jan 2013 04:24:53 +0000 (23:24 -0500)
committerRyan Lortie <desrt@desrt.ca>
Tue, 15 Jan 2013 04:31:59 +0000 (23:31 -0500)
commit5d42fdd068bddaaf7e42b7aaca2c15e931ebf3c5
treec0d7a9e960a1ad6dbb522818ee9b04e54c8cf18c
parent7e00f381916cab6ba13f1d430310802cf6b9c726
visibility: Use a separate CFLAGS variable

We only want to control the default visibility for our five main
installable libraries: libglib, libgthread, libgmodule, libgobject,
libgio.  We should therefore only set -fvisibility=hidden when building
those.

Use a separate substitution variable for this purpose.

Using CFLAGS directly leads to some modules built in testcases not
exporting their symbols (and then the tests fail).  It also affects the
fam file monitoring module.

Colin had originally done it this way in his visibility patch series but
I failed to understand why so I didn't copy it.  Now I do.

Also: revert changes made to two testcases in an attempt to work around
this issue.

https://bugzilla.gnome.org/show_bug.cgi?id=691756
configure.ac
gio/Makefile.am
gio/tests/resourceplugin.c
glib/Makefile.am
gmodule/Makefile.am
gobject/Makefile.am
gthread/Makefile.am
tests/libmoduletestplugin_a.c
tests/libmoduletestplugin_b.c
tests/module-test.c