mplayer options patch

[Anonymous]

Anton Shterenlikht <mexas at bristol.ac.uk> writes:

> On Fri, Jan 07, 2011 at 03:00:39PM +0300, Anonymous wrote:
>> Anton Shterenlikht <mexas at bristol.ac.uk> writes:
>> 
>> >>   $ make config
>> >>   "Makefile", line 16: Malformed conditional (${ARCH} == "i386" || ${ARCH} == "amd64" || ${ARCH} == "powerpc")
>> >>   "Makefile", line 18: if-less else
>> >>   "Makefile", line 20: if-less endif
>> >>   make: fatal errors encountered -- cannot continue
>> >
>> > I put Eygene's snippet after .include <bsd.port.pre.mk>,
>> > see my previous post. It seems to work.
>> 
>> bsd.port.pre.mk starts after OPTIONS were already parsed.
>> Try to check
>> 
>>   $ make rmconfig
>>   $ make -V WITH_RTCPU ARCH=sparc64
>>   true
>
> # make rmconfig
> ===> Removing user-configured options for mplayer-1.0.r20101106
> rmdir: /var/db/ports/mplayer: Directory not empty
> *** Error code 1 (ignored)
> # make -V WITH_RTCPU ARCH=sparc64
>
> # make -V WITH_RTCPU ARCH=ia64
>
> # 

My bad, I forgot to remove RTCPU when added the ifdef.

  $ make -V WITH_RTCPU ARCH=sparc64

  $ make -V WITHOUT_RTCPU ARCH=sparc64

  $ make -V CONFIGURE_ARGS ARCH=sparc64 | fgrep run
  ... --enable-runtime-cpudetection ... ${_LATE_CONFIGURE_ARGS}

It's not only for sparc64, there is no WITH/WITHOUT_RTPCU for i386.
But the configure option still works

  .if !defined(WITHOUT_RTCPU)
  CONFIGURE_ARGS+=--enable-runtime-cpudetection
  .endif