GNOME Bugzilla – Bug 661781
Configure option for allowing deprecated symbols
Last modified: 2011-10-27 15:53:05 UTC
Clutter-GST doesn't build against current git master of glib because it unconditionally defines G_DISABLE_DEPRECATED and GLib has changed the way GMutex works. Here are some patches to make it have a configure option to optionally remove that define for people who aren't clutter-gst developers and just want the sodding thing to build.
Created attachment 199016 [details] [review] autogen: Honour the NOCONFIGURE environment variable Most autogen scripts don't run the configure step if there is a NOCONFIGURE environment variable. This is quite useful if you are trying to do an out-of-tree build.
Created attachment 199017 [details] [review] configure: Use the release status define to determine default flags The --enable-debug option had a special m4_define to set the default value depending on whether the minor version of the release number suggests this is a git build. This patch changes it to use the clutter_gst_release_status define instead so that it by default is enabled only for git builds. The --enable-maintainer-flags option now uses this too. The help strings now have some added square brackets otherwise m4 doesn't seem to expand the macro properly.
Created attachment 199018 [details] [review] configure: Add a --enable-deprecated option This adds an option to enable deprecated symbols in dependant libraries. By default this is enabled on non-git builds. This makes it easier to build releases whilst steal annoying developers enough to fix the deprecations. The MAINTAINER_CFLAGS and DEBUG_CFLAGS substitutions have been merged into a single CLUTTER_GST_EXTRA_CFLAGS option which the deprecated option now also uses.
now that we get compiler warnings for deprecated symbols, there is no particularly good reason to ever define G_DISABLE_DEPRECATED for detecting use of deprecated symbols. You can still break your build if you combine -Wdeprecated-declarations with -Werror, if that's your desire.
Comment on attachment 199018 [details] [review] configure: Add a --enable-deprecated option I just removed the *DISABLE_DEPRECATED defines as we now get gcc warnings