Date: Fri, 15 Jul 2011 08:57:44 +0300 From: Ion-Mihai Tetcu <itetcu@FreeBSD.org> To: Juergen Lock <nox@jelal.kn-bremen.de>,freebsd-multimedia@FreeBSD.org Subject: Re: Fixing skype multimedia/linux_v4l2wrapper dependency on 8 Message-ID: <065a6e78-49d9-4431-b557-7b486c92c32f@email.android.com> In-Reply-To: <20110714205415.GA3304@triton8.kn-bremen.de> References: <20110714205415.GA3304@triton8.kn-bremen.de>
next in thread | previous in thread | raw e-mail | index | archive | help
Juergen Lock <nox@jelal.kn-bremen.de> wrote: >Hi! > > Installing the skype ports with VIDEO knob on currently fails on >up-to-date stable/8 because the multimedia/linux_v4l2wrapper port >now detects it is not needed anymore (my patch has been committed >to stable a while ago.) So I now made a patch that extracts the >check whether the code is in the kernel into a seperate Makefile.sub >that Linux ports that need V4L2 can include, and changed the skype >ports to do just that. Does this look good and ok to commit? > >Index: net-im/skype/Makefile >=================================================================== >RCS file: /home/pcvs/ports/net-im/skype/Makefile,v >retrieving revision 1.60 >diff -u -p -r1.60 Makefile >--- net-im/skype/Makefile 12 May 2011 08:15:48 -0000 1.60 >+++ net-im/skype/Makefile 14 Jul 2011 16:43:55 -0000 >@@ -46,10 +46,11 @@ USE_LINUX_APPS+=dri > .endif > > .ifdef WITH_VIDEO >+. include "${PORTSDIR}/multimedia/linux_v4l2wrapper-kmod/Makefile.sub" > RUN_DEPENDS+= webcamd>0:${PORTSDIR}/multimedia/webcamd > USE_LINUX_APPS+=libv4l > SUB_LIST+= VIDEO="LD_PRELOAD=/usr/lib/libv4l/v4l2convert.so " >-. if ${OSVERSION} < 900036 >+. if ${LINUX_V4L2_INBASE} != "1" >RUN_DEPENDS+= linux_v4l2wrapper-kmod>=1.0:${PORTSDIR}/multimedia/linux_v4l2wrapper-kmod > . endif > .else >Index: net-im/skype-devel/Makefile >=================================================================== >RCS file: /home/pcvs/ports/net-im/skype-devel/Makefile,v >retrieving revision 1.50 >diff -u -p -r1.50 Makefile >--- net-im/skype-devel/Makefile 17 May 2011 07:56:19 -0000 1.50 >+++ net-im/skype-devel/Makefile 14 Jul 2011 16:43:55 -0000 >@@ -45,10 +45,11 @@ USE_LINUX_APPS+=dri > .endif > > .ifdef WITH_VIDEO >+. include "${PORTSDIR}/multimedia/linux_v4l2wrapper-kmod/Makefile.sub" > RUN_DEPENDS+= webcamd>0:${PORTSDIR}/multimedia/webcamd > USE_LINUX_APPS+=libv4l > SUB_LIST+= VIDEO="LD_PRELOAD=/usr/lib/libv4l/v4l2convert.so " >-. if ${OSVERSION} < 900036 >+. if ${LINUX_V4L2_INBASE} != "1" >RUN_DEPENDS+= linux_v4l2wrapper-kmod>=1.0:${PORTSDIR}/multimedia/linux_v4l2wrapper-kmod > . endif > .else >Index: multimedia/linux_v4l2wrapper-kmod/Makefile >=================================================================== >RCS file: /home/pcvs/ports/multimedia/linux_v4l2wrapper-kmod/Makefile,v >retrieving revision 1.3 >diff -u -p -r1.3 Makefile >--- multimedia/linux_v4l2wrapper-kmod/Makefile 10 Jun 2011 19:07:36 >-0000 1.3 >+++ multimedia/linux_v4l2wrapper-kmod/Makefile 14 Jul 2011 16:43:55 >-0000 >@@ -38,18 +38,9 @@ MAKE_ENV+= SYSDIR="${SYSDIR}" > > CFLAGS+= ${DEBUG_FLAGS} > >-# try to avoid child processes when finding out if already in the >kernel >-.if ${OSVERSION} > 900036 || (${OSVERSION} < 900000 && ${OSVERSION} > >802508) >-INBASE= 1 >-.else >-.if ${OSVERSION} == 900036 || ${OSVERSION} == 802508 >-INBASE!= ($(SYSCTL) -n kern.features.linuxulator_v4l2 2>/dev/null || >true) >-.else >-INBASE= 0 >-.endif >-.endif >+.include "${.CURDIR}/Makefile.sub" > >-.if ${INBASE} == "1" >+.if ${LINUX_V4L2_INBASE} == "1" > IGNORE= is already in kernel > .else > .if !exists(${SYSDIR}/Makefile) >Index: multimedia/linux_v4l2wrapper-kmod/Makefile.sub >@@ -0,0 +1,12 @@ >+# $FreeBSD$ >+ >+# try to avoid child processes when finding out if already in the >kernel >+.if ${OSVERSION} > 900036 || (${OSVERSION} < 900000 && ${OSVERSION} > >802508) >+LINUX_V4L2_INBASE= 1 >+.else >+.if ${OSVERSION} == 900036 || ${OSVERSION} == 802508 >+LINUX_V4L2_INBASE!= ($(SYSCTL) -n kern.features.linuxulator_v4l2 >2>/dev/null || true) >+.else >+LINUX_V4L2_INBASE= 0 >+.endif >+.endif I'm in vacation until monday, please commit if you have the time. Thanks. -- Sent from my Android phone with K-9 Mail. Please excuse my brevity.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?065a6e78-49d9-4431-b557-7b486c92c32f>