[kde-freebsd] [patch] USB video support in KDE4 + sane + more under FreeBSD

[Hans Petter Selasky]

On Sunday 15 November 2009 00:07:38 Raphael Kubo da Costa wrote:
> On Sunday 11 October 2009 06:58:11 Hans Petter Selasky wrote:
> > Hi,
> >
> > I was compiling the latest version of KDE4 on my box and I noticed it
> >  picked up my installed libv4l2. The only problem is that you need some
> >  patches before everything is compiling :-)
> >
> > About libv4lX on FreeBSD, please see:
> >
> > http://lists.freebsd.org/pipermail/freebsd-emulation/2009-June/006231.htm
> >l
> >
> > Only USB webcams supported yet.
> >
> > After installation manually add -lv4lxdrivers to all the libv4l*.pc
> > files! I will try to get this included into my ulinux distribution soon.
> >
> > Before:
> > Libs: -L${libdir} -lv4lconvert
> > After:
> > Libs: -L${libdir} -lv4lconvert -lv4lxdrivers
> >
> > /usr/local/lib/pkgconfig/libv4lconvert.pc
> > /usr/local/lib/pkgconfig/libv4l2.pc
> > /usr/local/lib/pkgconfig/libv4l1.pc
> >
> > Then patch KDE4 like this:
> >
> > ---
> > 
> > /usr/ports/net/kdenetwork4/work/kdenetwork-4.3.1/kopete/libkopete/avdevic
> >e /videodevice.h.orig       2009-10-11 11:41:54.000000000 +0200 +++
> > /usr/ports/net/kdenetwork4/work/kdenetwork-4.3.1/kopete/libkopete/avdevic
> >e /videodevice.h    2009-10-11 11:44:34.000000000 +0200 @@ -62,6 +62,13 @@
> >
> >  #endif // __linux__
> >
> > +#if defined(__FreeBSD__) && defined(ENABLE_AV)
> > +#include <linux/videodev.h>
> > +#ifdef HAVE_LIBV4L2
> > +#include <libv4l2.h>
> > +#endif // HAVE_V4L2
> > +#endif
> > +
> >  #include <qstring.h>
> >  #include <qfile.h>
> >  #include <qimage.h
> >
> > And don't forget to chown /dev/usb/X.Y.* [webcam USB device] to the user
> >  running the webcam application.
> >
> > --HPS
>
> Hi there,
>
> I was thinking of committing the KDE part upstream. Is it OK to commit it
> like this (is it expected to work with 7.x too, for example)?
>
> Cheers,
> Raphael

Hi,

If libv4l2 is not present in the system, then adding those includes is not 
harmful, I think. Though I have not tried on FreeBSD 7.

BTW: The last:

> > +#endif // HAVE_V4L2

Should be:

> > +#endif // HAVE_LIBV4L2

--HPS