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 451970 - Subparse requires HTML parser
Subparse requires HTML parser
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
0.10.13
Other All
: Normal normal
: 0.10.15
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-06-28 18:06 UTC by Daniel Díaz
Modified: 2007-08-30 06:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Use XML instead of HTML parser (2.07 KB, patch)
2007-06-28 18:11 UTC, Daniel Díaz
none Details | Review
Check for HTML support in libxml2 (1.24 KB, patch)
2007-07-16 19:31 UTC, Daniel Díaz
committed Details | Review

Description Daniel Díaz 2007-06-28 18:06:44 UTC
There is no ./configure swtich to avoid compiling the subparse plug-in, and with a non-HTML libxml2, compiling subparse would fail with:

samiparse.c:48: error: syntax error before "htmlParserCtxtPtr"
samiparse.c:48: warning: no semicolon at end of struct or union
samiparse.c:54: error: syntax error before '}' token
samiparse.c: In function `sami_context_push_state':
samiparse.c:65: error: dereferencing pointer to incomplete type
samiparse.c: In function `sami_context_pop_state':
samiparse.c:72: error: dereferencing pointer to incomplete type

...and so on.
Comment 1 Daniel Díaz 2007-06-28 18:11:19 UTC
Created attachment 90821 [details] [review]
Use XML instead of HTML parser
Comment 2 Daniel Díaz 2007-06-28 19:07:32 UTC
Somebody please test the patch because I have no means of using subparse in my environment.
Comment 3 Young-Ho Cha 2007-07-14 05:13:09 UTC
Sami subtitle format is HTML, not XML.

Just check libxml2 whether support html and disable samiparse feature.
Comment 4 Daniel Díaz 2007-07-16 19:31:38 UTC
Created attachment 91869 [details] [review]
Check for HTML support in libxml2
Comment 5 Stefan Sauer (gstreamer, gtkdoc dev) 2007-08-30 06:58:13 UTC
2007-08-30  Stefan Kost  <ensonic@users.sf.net>

	Patch by: Daniel Díaz  <yosoy@danieldiaz.org>

	* configure.ac:
	* gst/Makefile.am:
	  Check if libxml provides HTML parser which subparse needs.
	  Fixes #451970.