diff options
| author | Ryan Lortie <desrt@desrt.ca> | 2013-01-14 23:24:53 -0500 |
|---|---|---|
| committer | Ryan Lortie <desrt@desrt.ca> | 2013-01-14 23:31:59 -0500 |
| commit | 5d42fdd068bddaaf7e42b7aaca2c15e931ebf3c5 (patch) | |
| tree | c0d7a9e960a1ad6dbb522818ee9b04e54c8cf18c /gmodule | |
| parent | 7e00f381916cab6ba13f1d430310802cf6b9c726 (diff) | |
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
Diffstat (limited to 'gmodule')
| -rw-r--r-- | gmodule/Makefile.am | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/gmodule/Makefile.am b/gmodule/Makefile.am index 19737fa3f..92b844a41 100644 --- a/gmodule/Makefile.am +++ b/gmodule/Makefile.am @@ -62,6 +62,7 @@ gmodule_win32_res = gmodule-win32-res.o gmodule_win32_res_ldflag = -Wl,$(gmodule_win32_res) endif +libgmodule_2_0_la_CFLAGS = $(AM_CFLAGS) $(GLIB_HIDDEN_VISIBILITY_CFLAGS) libgmodule_2_0_la_SOURCES = gmodule.c libgmodule_2_0_la_LDFLAGS = $(GLIB_LINK_FLAGS) \ $(gmodule_win32_res_ldflag) \ |