On Thu, Mar 24, 2011 at 12:11:17PM +0000, Edd Barrett wrote:
> On Thu, Mar 24, 2011 at 07:05:33AM +0100, Antoine Jacoutot wrote:
> > Why is this needed?
>
> It's not. Perhaps a legacy. Removed.
>
> > I think you are missing some markers. Like:
> > @pkgpath x11/mplayer,sdl
> > @pkgpath x11/mplayer,sdl,aa
>
> I have done the "recently seen" flavors. As we decided on ICB, there are
> way too many legacy flavors to enumerate each and every combination of
> them all.
>
>
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/x11/mplayer/Makefile,v
> retrieving revision 1.181
> diff -u -p -u -r1.181 Makefile
> --- Makefile 12 Feb 2011 10:49:36 -0000 1.181
> +++ Makefile 24 Mar 2011 12:08:15 -0000
> @@ -5,19 +5,18 @@ ONLY_FOR_ARCHS = alpha amd64 arm i386 mi
>
> COMMENT = movie player supporting many formats
>
> -V = 20101024
> -V_DASHED = 2010-10-24
> +V = 20110309
> +FFMPEG_V = 20110306
>
> SUBST_VARS += PREFIX CONFDIR
>
> +# Distfiles must be hand-rolled, see README
> N = mplayer
> -DISTNAME = mplayer-export-snapshot-${V}
> +DISTNAME = mplayer-${V}
> +
> +DISTFILES = ${DISTNAME}${EXTRACT_SUFX}
i think you don't need that anymore since the extra ffmpeg distfile was
taken out
> +# To do dynamic ffmpeg linking, you also need the ffmpeg sources inside
> +# the build tree for some obscure reason. This should retrieve the same
> +# distfile as graphics/ffmpeg.
Comment is wrong, no ?
> -MD5 (mplayer-export-snapshot-20101024.tar.bz2) = F797FlNAF1eu01Goo2VXNw==
> -RMD160 (mplayer-export-snapshot-20101024.tar.bz2) =
> S9533qXXp0k7Y5TXnXHbk4yfRBs=
> -SHA1 (mplayer-export-snapshot-20101024.tar.bz2) =
> iGBuBIpI6dKVOxdmJ4HKyatTHUM=
> -SHA256 (mplayer-export-snapshot-20101024.tar.bz2) =
> BbFJxSzFbQ2CpHSGr0b5UAGLJ0wu8VqHCMvBsvy5uJE=
> -SIZE (mplayer-export-snapshot-20101024.tar.bz2) = 9998106
> +MD5 (ffmpeg-git-20110306.tar.gz) = sfTm9NP2uKSM6gBhiuAfoA==
> +MD5 (mplayer-20110309.tar.gz) = EU2sXltwhr4VhUplf8qsXw==
> +RMD160 (ffmpeg-git-20110306.tar.gz) = ec7bfWXTeMQKDoHSTmnDQWZ/0Pk=
> +RMD160 (mplayer-20110309.tar.gz) = QlDlgZxUyawqSdBWC871ys/QoRA=
> +SHA1 (ffmpeg-git-20110306.tar.gz) = s6wHaPwZFMZRaaT+/LP+HiNaSCk=
> +SHA1 (mplayer-20110309.tar.gz) = AMaoe0ip6hI2EyYdtOf6uHgcgio=
> +SHA256 (ffmpeg-git-20110306.tar.gz) =
> Av5pUZYm1q+shteP/tHYeHgF98bB3whT20zIyq1x5L8=
> +SHA256 (mplayer-20110309.tar.gz) =
> yeswxU3WOlLg5FnCJQJL5hLLopvSp543ovqjPN4FGSI=
> +SIZE (ffmpeg-git-20110306.tar.gz) = 4865660
> +SIZE (mplayer-20110309.tar.gz) = 6874760
Regen distinfo, since ffmpeg-git distfile is out.
> + ffmpeg's ogg demuxer is broken for ogg streams. If mplayer says "cache
> + not filling" when a song changes, you have been bitten by this bug.
> + You can try '-demuxer ogg' as a workaround (or try using ogg123 from
> + the vorbis-tools package).
That's a really annoying bug.. was it reported upstream, being worked
on, tracked somewhere in a bug report ? Have you checked other oses to
see if it happens/is fixed there ?
Landry
