gst-plugins-base-0.10.32: GST_DISABLE_XML=1 doesn't work in gstsubparse.c

[bcxa sz]

Hi Tim,

I attached my config.h here and I didn't see the defination.

All the Makefile and configure.h is generated automatically. And I
have a/v working on my embedded system with compiled gstreamer
library.

Just had this problem when I try to add subtitle support.

best regards

BCXA

On Tue, Aug 16, 2011 at 5:48 PM, Tim-Philipp Müller <t.i.m at zen.co.uk> wrote:
> On Tue, 2011-08-16 at 17:42 +0800, bcxa sz wrote:
>
> Hi,
>
>> I have cross compiled gstreamer on embedded system. And I have below
>> flags in gst-plugins-base-0.10.32/gst/subparse/Makefile:
>
> Maybe you should try the latest release, which is 0.10.35.
>
>> GST_DISABLE_ALLOC_TRACE = 1
>> GST_DISABLE_GST_DEBUG = 1
>> GST_DISABLE_LOADSAVE = 1
>> GST_DISABLE_PARSE = 1
>> GST_DISABLE_PLUGIN = 1
>> GST_DISABLE_REGISTRY = 1
>> GST_DISABLE_TRACE = 1
>> GST_DISABLE_XML = 1
>
> You are setting Makefile variables here. These have nothing to do with
> #ifdef and #ifndef.
>
>> And look at the gstsubparse.c:
>> #ifndef GST_DISABLE_XML
>>     case GST_SUB_PARSE_FORMAT_SAMI:
>>       self->parse_line = parse_sami;
>>       sami_context_init (&self->state);
>>       return gst_caps_new_simple ("text/x-pango-markup", NULL);
>> #endif
>>
>> This piece of code is assumed not to get compiled. But this is not
>> true and it still get compiled.
>>
>> And it seemed that GST_DISABLE_XML flag is not given to compiler with -D.
>
> Those defines would usually be in config.h in the top-level directory
> (created and set by configure dependent on whether libxml2 development
> files + headers have been found or not).
>
> Cheers
>  -Tim
>
>
> _______________________________________________
> gstreamer-devel mailing list
> gstreamer-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/gstreamer-devel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: config.h
Type: text/x-chdr
Size: 10880 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/gstreamer-devel/attachments/20110816/dbc58629/attachment-0001.h>