GNOME Bugzilla – Bug 543286
deinterlace2 doesn't build outside gcc
Last modified: 2008-07-18 08:33:51 UTC
Patch attached to fix the deinterlace2 build with Forte, which probably fixes it for other non-gcc compilers too.
Created attachment 114665 [details] [review] fix forte build
Created attachment 114667 [details] [review] fix forte build remove a little whitespace noise in the first patch.
Ok, I've fixed this a bit different now. tomsmocomp is now disabled for this release until I've ported it and the bugs are fixed. Also the plugin is now built on all architectures but the inline assembly is only built if we build on x86 or amd64 and gcc inline assembly is supported. Can you confirm that this works on forte too this way? 2008-07-18 Sebastian Dröge <sebastian.droege@collabora.co.uk> * configure.ac: * gst/deinterlace2/Makefile.am: * gst/deinterlace2/gstdeinterlace2.c: (gst_deinterlace2_methods_get_type), (gst_deinterlace2_set_method), (gst_deinterlace2_class_init), (gst_deinterlace2_init): * gst/deinterlace2/gstdeinterlace2.h: * gst/deinterlace2/tvtime/greedy.c: (gst_deinterlace_method_greedy_l_class_init): * gst/deinterlace2/tvtime/greedyh.c: (gst_deinterlace_method_greedy_h_class_init): * gst/deinterlace2/tvtime/vfir.c: (gst_deinterlace_method_vfir_class_init): Disable the tomsmocomp algorithm for this release as it's buggy and has no C implementation yet. Build the deinterlace2 plugin on all architectures but still mark it as experimental. Build the x86 inline assembly only if GCC inline assembly is supported and only on x86 or amd64. Fixes bug #543286.