mplayer options patch

[Eygene Ryabinkin]

Fri, Jan 07, 2011 at 01:32:39PM +0000, Anton Shterenlikht wrote:
> I don't think I need to change any other options from 
> the current default to make it build and run on my ia64.
> 
> So if this patch seems reasonable, can somebody submit it?
> 
> many thanks
> anton
> 
> 
> --- Makefile	2010-12-02 09:18:00.000000000 +0000
> +++ Makefile.new	2011-01-07 11:16:56.000000000 +0000
> @@ -13,7 +13,6 @@
>  .include "${.CURDIR}/Makefile.shared"
>  
>  OPTIONS=	DEBUG "Include debug symbols in mplayer's binaries" off
> -OPTIONS+=	RTCPU "Allow runtime CPU detection" on
>  OPTIONS+=	OCFLAGS "Use optimized compiler flags" on
>  OPTIONS+=	IPV6 "Include inet6 network support" on
>  OPTIONS+=	X11 "Enable X11 support for mplayer's video output" on
> @@ -75,6 +74,12 @@
>  BROKEN=		Does not compile on sparc64
>  .endif
>  
> +.if ${ARCH} == "i386" || ${ARCH} == "amd64" || ${ARCH} == "powerpc"
> +OPTIONS+=       RTCPU "Allow runtime CPU detection" on
> +.else
> +OPTIONS+=       RTCPU "Allow runtime CPU detection" off
> +.endif

Having thinked about it for a while, I wonder if we need to have
RTCPU knob on non-{i386,amd64,powerpc}?  From the one hand, the
knob isn't functional, so it is redundant, because mplayer will
refuse to build.  On the other hand, once mplayer will gain the
ability to make run-time CPU detection on other platforms, this
knob will be usable again and it will be good to add it to the
relevant architectures.

I think that this should be left to the maintainers decision.
Please, submit a PR that describes the problem and contains
the patch.  If submitting PR is something you hadn't done before,
please, take a look at
  http://www.freebsd.org/doc/en_US.ISO8859-1/articles/problem-reports/article.html

You can add me to the X-GNATS-Notify header -- I'll take the PR
and once/if Thomas will approve it, I'll commit the modifications.
-- 
Eygene Ryabinkin                                        ,,,^..^,,,
[ Life's unfair - but root password helps!           | codelabs.ru ]
[ 82FE 06BC D497 C0DE 49EC  4FF0 16AF 9EAE 8152 ECFB | freebsd.org ]
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-ports/attachments/20110107/51de611e/attachment.pgp