GNOME Bugzilla – Bug 623845
[P12] Use -Bsymbolic
Last modified: 2010-07-10 00:59:15 UTC
This is one of the preparative tasks for GTK+3: http://live.gnome.org/GTK+/3.0/Tasks#P12 This is already done in Glib.
Created attachment 165485 [details] [review] Add check for -Bsymbolic-functions linker flag
Created attachment 165486 [details] [review] Use -Bsymbolic-functions
These patches look ok to me, but we should also get the cleanup parts done, ie remove the disable-visiblity stuff from configure, drop the gtkalias.h etc generation, drop the #includes from all sources.
Created attachment 165562 [details] [review] gdk/: fully remove gdkalias hacks
Created attachment 165567 [details] [review] gtk/: fully remove gtkalias hacks
Ryan pointed out on irc that the glib configure has meanwhile grown a --disable-bsymbolic option that we should copy over as well.
Other than that, looks fine
Comment on attachment 165485 [details] [review] Add check for -Bsymbolic-functions linker flag committed an improved version with the new --disable-Bsymbolic option commit e4e477161ae617a35528c0659f00284f4c80c372
Comment on attachment 165486 [details] [review] Use -Bsymbolic-functions commit 7cabad6b77df866b59b8b98b8c8efaa6e89fd590
Comment on attachment 165562 [details] [review] gdk/: fully remove gdkalias hacks commit 404e7d0e0013355549578db5c3981943f536c2b9
Comment on attachment 165567 [details] [review] gtk/: fully remove gtkalias hacks commit 0a07e9733bb259598a09515a3e4cdbcda5feef57
This problem has been fixed in the development version. The fix will be available in the next major software release. Thanks for your review.