pkg/41729: pkgsrc -current multimedia/ffmpeg missing sdl+ffplay

[Matthew Mondor <mm_lists%pulsar-zone.net@localhost>]

>Number:         41729
>Category:       pkg
>Synopsis:       pkgsrc -current multimedia/ffmpeg missing sdl+ffplay
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Tue Jul 14 22:45:00 +0000 2009
>Originator:     Matthew Mondor
>Release:        NetBSD 5.0_STABLE
>Organization:
>Environment:
System: NetBSD ginseng.xisop 5.0_STABLE NetBSD 5.0_STABLE (GENERIC_MM) #1: Thu 
Jul 9 01:57:16 EDT 2009 
root%behemoth.xisop@localhost:/usr/obj/sys/arch/i386/compile/GENERIC_MM i386
Architecture: i386
Machine: i386
>Description:
        ffmpeg supports SDL as and includes a player utility based on SDL
        called ffplay.  This is usually built when using the sdl option.

        Although this still works on pkgsrc-2009Q1, on -current this support
        is missing, but is trivial to restore.  In fact the option stub
        already exists but is missing from PKG_SUPPORTED_OPTIONS and the
        PLIST lacks the conditional ffplay binary and manual page.
>How-To-Repeat:
>Fix:

Index: PLIST
===================================================================
RCS file: /nfs/ginseng/home/data/cvsup/netbsd/pkgsrc/multimedia/ffmpeg/PLIST,v
retrieving revision 1.13
diff -u -r1.13 PLIST
--- PLIST       14 Jun 2009 18:08:03 -0000      1.13
+++ PLIST       14 Jul 2009 22:08:07 -0000
@@ -1,6 +1,8 @@
 @comment $NetBSD: PLIST,v 1.13 2009/06/14 18:08:03 joerg Exp $
 bin/ffmpeg
 bin/ffserver
+${PLIST.sdl}bin/ffplay
+${PLIST.sdl}man/man1/ffplay.1
 include/libavcodec/avcodec.h
 include/libavcodec/opt.h
 include/libavcodec/vdpau.h
Index: options.mk
===================================================================
RCS file: 
/nfs/ginseng/home/data/cvsup/netbsd/pkgsrc/multimedia/ffmpeg/options.mk,v
retrieving revision 1.12
diff -u -r1.12 options.mk
--- options.mk  12 Jun 2009 16:25:34 -0000      1.12
+++ options.mk  14 Jul 2009 21:38:34 -0000
@@ -3,7 +3,7 @@
 # Global and legacy options
 
 PKG_OPTIONS_VAR=       PKG_OPTIONS.ffmpeg
-PKG_SUPPORTED_OPTIONS= theora xvid faad faac x264
+PKG_SUPPORTED_OPTIONS= sdl theora xvid faad faac x264
 PKG_SUGGESTED_OPTIONS= theora xvid x264
 #PKG_OPTIONS_OPTIONAL_GROUPS=  aac-decoder
 #PKG_OPTIONS_GROUP.aac-decoder=        faad faac