Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 29 Aug 2015 21:48:42 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-ports-bugs@FreeBSD.org
Subject:   [Bug 202404] [MAINTAINER] multimedia/mplayer2: updates to Makefile
Message-ID:  <bug-202404-13-PYYIMFebGl@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-202404-13@https.bugs.freebsd.org/bugzilla/>
References:  <bug-202404-13@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D202404

Jan Beich <jbeich@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|In Progress                 |Closed

--- Comment #41 from Jan Beich <jbeich@FreeBSD.org> ---
Thanks for patience. Committed. I've fixed the rest myself.

> -.if !defined(PACKAGE_BUILDING)
>  OPTIONS_DEFINE_i386+=3D	WIN32
>  OPTIONS_DEFAULT_i386+=3D	WIN32
> -.endif

win32-codecs is a RESTRICTED package thus will make any port that
depends on it broken due to unsatisfied dependency. The check
originates from bug 115170 for mplayer.

_OFF option helper won't trigger if the option isn't defined via
_DEFINE*, _SINGLE, etc. That's why I've moved OPTIONS_DEFINE_i386
outside of PACKAGE_BUILDING in comment 2. Thus package cluster
makes sure to provide --disable-win32dll when building for i386,
the only architecture where configure tries to auto-enable it.

> OPTIONS_DEFAULT=3D... RTCPU ...

Did you check on either arm*, aarch64, mips*, ia64, sparc64 ?
--enable-runtime-cpudetection is only supported on x86 and ppc
which translates to i386, amd64, powerpc, powerpc64 on FreeBSD.
On other architectures configure calls die() i.e., exits with
error message quoted in comment 37.

> -V4L_LIBS=3D		-lv4l2
> +V4L_LIBS=3D		-lv4l1 -lv4l2

libv4l2 usage is inflicted by files/patch-stream-tvi_v4l2.c. Upstream
only uses <linux/videodev2.h> which is part of V4L2 API. Nothing in mplayer2
uses <linux/videodev.h> aka V4L1 API not to mention <libv4l1.h>.

Explain if you think otherwise.

> -.if ${PORT_OPTIONS:MREALPLAYER}
> -RUN_DEPENDS+=3D		realplay:${PORTSDIR}/multimedia/linux-realplayer
> -BUILD_DEPENDS+=3D		realplay:${PORTSDIR}/multimedia/linux-realplayer
> -.else
> -CONFIGURE_ARGS+=3D	--disable-real
> -.endif
> +REALPLAYER_BUILD_DEPENDS=3Drealplay:${PORTSDIR}/multimedia/linux-realpla=
yer
> +REALPLAYER_CONFIGURE_OFF=3D--disable-real

RUN_DEPENDS are lost in conversion.

> WITH_CDROM_DEVICE?=3D	/dev/cd0

This is still referenced.

> -.if defined(WITH_KERN_HZ)
> -DEFAULT_KERN_HZ=3D${WITH_KERN_HZ}
> -.else
> -DEFAULT_KERN_HZ=3D1024
> -.endif

Did you ignore the build error I reported in comment 37 ?

> -	@${REINPLACE_CMD} -e \
> -		's|irqp =3D 1024|irqp =3D ${DEFAULT_KERN_HZ}|' \
> -		${WRKSRC}/mplayer.c
> +		@${REINPLACE_CMD} -e \
> +			's|irqp =3D 1024|irqp =3D ${DEFAULT_KERN_HZ}|' \
> +			${WRKSRC}/mplayer.c

Pointless reindenting causes noise for |svn blame|.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-202404-13-PYYIMFebGl>