Fix v4l/v4l2 compilation problems
authorRonald S. Bultje <rbultje@ronald.bitfreak.net>
Sun, 9 Mar 2003 15:07:53 +0000 (15:07 +0000)
committerRonald S. Bultje <rbultje@ronald.bitfreak.net>
Sun, 9 Mar 2003 15:07:53 +0000 (15:07 +0000)
Original commit message from CVS:
Fix v4l/v4l2 compilation problems

configure.ac
sys/v4l2/gstv4l2element.h

index 3521b0b..0981b9b 100644 (file)
@@ -290,15 +290,36 @@ dnl this one is for hadess, no qcam when no nasm found
 ])
 
 dnl *** Video 4 Linux ***
+dnl for information about the header/define, see sys/v4l/gstv4lelement.h
 translit(dnm, m, l) AM_CONDITIONAL(USE_V4L, true)
 GST_CHECK_FEATURE(V4L, [Video 4 Linux], v4lsrc v4lmjpegsrc v4lmjpegsink, [
-  AC_CHECK_DECL(VID_TYPE_MPEG_ENCODER, HAVE_V4L="yes", HAVE_V4L="no", [#include <linux/videodev.h>])
+  AC_CHECK_DECL(VID_TYPE_MPEG_ENCODER, HAVE_V4L="yes", HAVE_V4L="no", [
+#include <sys/types.h>
+#define _LINUX_TIME_H
+#include <linux/videodev.h>
+  ])
 ])
 
 dnl *** Video 4 Linux 2***
+dnl for information about the header/define, see sys/v4l2/gstv4l2element.h
 translit(dnm, m, l) AM_CONDITIONAL(USE_V4L2, true)
 GST_CHECK_FEATURE(V4L2, [Video 4 Linux 2], v4l2src, [
-  AC_CHECK_HEADER(linux/videodev2.h, HAVE_V4L2="yes", HAVE_V4L2="no")
+  AC_MSG_CHECKING([Checking for uptodate v4l2 installation])
+  AC_TRY_COMPILE([
+#include <sys/types.h>
+#include <linux/types.h>
+#define _LINUX_TIME_H
+#include <linux/videodev2.h>
+#if defined(V4L2_MAJOR_VERSION) || defined(V4L2_MINOR_VERSION)
+#error too early v4l2 version or no v4l2 at all
+#endif
+  ], [
+return 0;
+  ], [ HAVE_V4L2="yes" && AC_MSG_RESULT(yes)],
+     [ HAVE_V4L2="no"  && AC_MSG_RESULT(no) &&
+       AC_CHECK_HEADER(linux/videodev2.h,
+                       [ AC_MSG_WARN([video4linux2 headers were found, but they're old. Please update v4l2 to compile the v4l2 plugins])],
+                       [ AC_MSG_WARN([video4linux2 was not found])])])
 ])
 
 dnl *** Video CD ***
index 0e46203..76ad04d 100644 (file)
 #define __GST_V4L2ELEMENT_H__
 
 #include <gst/gst.h>
+
+/* Because of some really cool feature in video4linux1, also known as
+ * 'not including sys/types.h and sys/time.h', we had to include it
+ * ourselves. In all their intelligence, these people decided to fix
+ * this in the next version (video4linux2) in such a cool way that it
+ * breaks all compilations of old stuff...
+ * The real problem is actually that linux/time.h doesn't use proper
+ * macro checks before defining types like struct timeval. The proper
+ * fix here is to either fuck the kernel header (which is what we do
+ * by defining _LINUX_TIME_H, an innocent little hack) or by fixing it
+ * upstream, which I'll consider doing later on. If you get compiler
+ * errors here, check your linux/time.h && sys/time.h header setup.
+ */
 #include <sys/types.h>
 #include <linux/types.h>
+#define _LINUX_TIME_H
 #include <linux/videodev2.h>