From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-11-30 14:41:59
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified Summary: deinterlace plugin does not build universal on Mac OS Classification: Desktop Product: GStreamer Version: unspecified OS/Version: Mac OS Status: UNCONFIRMED Severity: normal Priority: Normal Component: gst-plugins-good AssignedTo: gst...@li... ReportedBy: jer...@fr... QAContact: gst...@li... GNOME target: --- GNOME version: --- The problem is that you're using configure-time checks for architecture rather than compile-time checks. In addition to the attached patch, you should probably move checks for SIMD out of configure as well using __MMX__ , __SSE__, __SSE2__, __SSE3__, etc --- make -C deinterlace make all-am CC libgstdeinterlace_la-tomsmocomp.lo /bin/sh ../../libtool --silent --tag=CC --tag=disable-static --mode=compile /usr/bin/gcc-4.2 -std=gnu99 -DHAVE_CONFIG_H -I. -I../.. -I/opt/local/include -D_REENTRANT -I/opt/local/include/gstreamer-0.10 -I/opt/local/include/glib-2.0 -I/opt/local/lib/glib-2.0/include -I/opt/local/include -I/opt/local/include/libxml2 -D_REENTRANT -I/opt/local/include/gstreamer-0.10 -I/opt/local/include/glib-2.0 -I/opt/local/lib/glib-2.0/include -I/opt/local/include -I/opt/local/include/libxml2 -I../../gst-libs -D_REENTRANT -I/opt/local/include/gstreamer-0.10 -I/opt/local/include/glib-2.0 -I/opt/local/lib/glib-2.0/include -I/opt/local/include -I/opt/local/include/libxml2 -DG_THREADS_MANDATORY -DG_DISABLE_CAST_CHECKS -Wall -Wdeclaration-after-statement -Wpointer-arith -Wmissing-declarations -Wmissing-prototypes -Wredundant-decls -Wwrite-strings -Wold-style-definition -Winit-self -Wmissing-include-dirs -Waddress -Wno-multichar -Wnested-externs -g -I/opt/local/include/orc-0.4 -O2 -arch x86_64 -arch i386 -c -o libgstdeinterlace_la-tomsmocomp.lo `test -f 'tvtime/tomsmocomp.c' || echo './'`tvtime/tomsmocomp.c /var/tmp//ccdRgmRy.s:11200:bad register name `%rbx' /var/tmp//ccdRgmRy.s:11201:bad register name `%rbx' /var/tmp//ccdRgmRy.s:11202:bad register name `%rcx' /var/tmp//ccdRgmRy.s:11203:bad register name `%rbx)' /var/tmp//ccdRgmRy.s:11204:bad register name `%rbx, %rcx)' /var/tmp//ccdRgmRy.s:11205:bad register name `%rdi' /var/tmp//ccdRgmRy.s:11206:bad register name `%rdi)' /var/tmp//ccdRgmRy.s:11207:bad register name `%rdx' /var/tmp//ccdRgmRy.s:11208:bad register name `%rbx, %rdx)' /var/tmp//ccdRgmRy.s:11209:bad register name `%rsi)' /var/tmp//ccdRgmRy.s:11210:bad register name `%rsi, %rcx)' /var/tmp//ccdRgmRy.s:11211:bad register name `%rdi,%rdx)' /var/tmp//ccdRgmRy.s:11212:bad register name `%rsi' /var/tmp//ccdRgmRy.s:11213:bad register name `%rdi' /var/tmp//ccdRgmRy.s:11214:bad register name `%rdx' -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-11-30 14:42:30
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified --- Comment #1 from Jeremy Huddleston <jer...@fr...> 2010-11-30 14:42:21 UTC --- Created an attachment (id=175535) View: https://bugzilla.gnome.org/attachment.cgi?id=175535 Review: https://bugzilla.gnome.org/review?bug=636134&attachment=175535 tvtime-universal.patch -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-11-30 14:53:49
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified Jeremy Huddleston <jeremyhu> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #175535|0 |1 is obsolete| | --- Comment #2 from Jeremy Huddleston <jer...@fr...> 2010-11-30 14:53:34 UTC --- (From update of attachment 175535) please ignore, this is not the correct diff. -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-11-30 15:26:07
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified --- Comment #3 from Jeremy Huddleston <jer...@fr...> 2010-11-30 15:25:57 UTC --- Created an attachment (id=175538) View: https://bugzilla.gnome.org/attachment.cgi?id=175538 Review: https://bugzilla.gnome.org/review?bug=636134&attachment=175538 universal.patch This patch fixes cpu (and some SIMD) detection. Endian checks should be removed from configure.ac as well. -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-12-11 16:06:05
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified Sebastian Dröge <slomo> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEEDINFO CC| |sl...@ci... --- Comment #4 from Sebastian Dröge <sl...@ci...> 2010-12-11 16:05:52 UTC --- Erm, no. There are configure time checks for these things for the simple reason that the preprocessor defines are in no way standardized and that you need different checks for different architectures. Instead of removing the configure checks it would be better to improve them to correctly work everywhere. What exactly do you mean by "build universal" btw? -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-12-11 16:22:02
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified Tim-Philipp Müller <t.i.m> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |t....@ze... --- Comment #5 from Tim-Philipp Müller <t....@ze...> 2010-12-11 16:21:48 UTC --- Well, if I'm not mistaken our configure stuff is quite broken in many respects - e.g. for some things we check we assume host system == target or assume if kernel is 64bit user space is 64 bit etc. -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-12-11 18:21:29
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified David Schleef <ds> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |RESOLVED Resolution| |WONTFIX --- Comment #6 from David Schleef <ds...@sc...> 2010-12-11 18:21:19 UTC --- "Universal build" means compiling the code for multiple architectures into a single .dylib file. It's a hack that OS/X does to compile for, say, x86_64 and x86, or previously, x86 and powerpc. It's really not possible to do this generically on something as big and complex as GStreamer. Notably, it fails on assembly code. It also fails for things like glibconfig.h, which is a different header on different architectures. Since it is fairly straightforward to combine the .dylib/.so files created by two separate builds (there's a tool on OS/X specifically for this), I have never considered it a reasonable goal to support universal building at the compiler level. -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-12-12 01:06:08
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified --- Comment #7 from Jeremy Huddleston <jer...@fr...> 2010-12-12 01:05:36 UTC --- Well your configure time checks are *WRONG* then because they only check one architecture. -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-12-12 01:07:11
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified --- Comment #8 from Jeremy Huddleston <jer...@fr...> 2010-12-12 01:06:55 UTC --- Those types of checks should be done at built time rather than configure time. Plenty of other autoconf based projects (pixman, cairo, X.org, ...) do these at build time rather than configure time. -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-12-12 01:08:14
|
https://bugzilla.gnome.org/show_bug.cgi?id=636134 GStreamer | gst-plugins-good | unspecified --- Comment #9 from Jeremy Huddleston <jer...@fr...> 2010-12-12 01:08:00 UTC --- And it doesn't fail on assembly code. You just need to properly #ifdef your code. Closing a bug and ignoring it does not make the problem go away. -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |