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 590841 - configure script check for libxml2 is incorrect
configure script check for libxml2 is incorrect
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
0.10.24
Other All
: Normal minor
: 0.10.25
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-08-05 12:08 UTC by Will Newton
Modified: 2009-08-06 09:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Will Newton 2009-08-05 12:08:52 UTC 
Please describe the problem:

The XML registry support has been removed, but the configure script still checks for it when deciding whether or not libxml2 is needed:

dnl libxml 2 is optional in some cases
if test "x$GST_DISABLE_LOADSAVE" = "xyes" -a "x$GST_DISABLE_REGISTRY" = "xyes"
then
  AC_MSG_NOTICE([XML registry and load/save are disabled, not checking for libxml2])
  GST_DISABLE_XML_DEFINE="#define GST_DISABLE_XML 1"
else


Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Tim-Philipp Müller 2009-08-05 12:39:52 UTC 
Fixed locally, will push as soon as core is unfrozen.
Comment 2 Tim-Philipp Müller 2009-08-06 08:25:30 UTC 
Hope that's what you meant, if not, please re-open, thanks!

 commit cc57c404fd722cfe47f337b645e26197baab4c97
 Author: Tim-Philipp Müller <tim.muller@collabora.co.uk>
 Date:   Wed Aug 5 13:25:33 2009 +0100

    configure.ac: fix libxml2 check, which is only needed for xml load/save now
    
    Since the registry doesn't use libxml2 any longer, it's no longer necessary
    to disable both xml load/save *and* the registry to get rid of the libxml2
    dependency, disabling just xml loading/saving is enough. Fixes #590841.
Comment 3 Will Newton 2009-08-06 09:23:45 UTC 
Yes, thanks for the quick turnaround!