After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 562168 - Good plugins' configury overloads --disable-debug
Good plugins' configury overloads --disable-debug
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal major
: 0.10.15
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-11-24 19:39 UTC by Branko Čibej
Modified: 2009-04-15 13:17 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Branko Čibej 2008-11-24 19:39:39 UTC
This one falls into the "obvious in retrospect" (headbang) category.

In gst-plugins-good, ./configure --help says:
[... stuff]
  --disable-debug         disable addition of -g debugging info
[... more stuff]
  --disable-debug         disable dependency-less debug plugin

Oops.

'./configure --disable-debug --with-plugins=debug' does what it's told, but certainly not what you'd expect. This is most frustrating.

This issue appears to have been introduced in 0.10.8 when configure.ac started using AG_GST_CHECK_PLUGIN for dependency-less plugins.

The only marginally sane change that I can think of would be to e.g., rename gst-plugins-good/gst/debug to gst-plugins-good/gst/gstdebug, so that the debug plugin's subdir name doesn't cause a conflict with the standard --disable-debug argument.
Comment 1 Stefan Sauer (gstreamer, gtkdoc dev) 2009-03-20 19:46:57 UTC
Thanks for the analysis, will discuss with the team and resolve before next release.
Comment 2 Stefan Sauer (gstreamer, gtkdoc dev) 2009-04-15 13:17:25 UTC
commit 7be792fa138dcb4634eb893074042b7440e56594
Author: Stefan Kost <ensonic@users.sf.net>
Date:   Wed Apr 15 16:03:27 2009 +0300

    debug: rename debug to debugutils to avoid clash with --disable-debug. Fixes #562168

commit 99fcc86ee441d34752b39e382014d5a8a5cb04b0
Author: Stefan Kost <ensonic@users.sf.net>
Date:   Wed Apr 15 15:43:04 2009 +0300

    debug: indent before renaming