[MPlayer-dev-eng] Re: MPlayer package for Debian

[Dominik 'Rathann' Mierzejewski]

On Monday, 16 January 2006 at 17:46, A Mennucc wrote:
> hi
> 
> I prepared three versions of 1.0pre7try2 , that can be accessed
> by adding the quoted lines in /etc/apt/sources.list
> 
>  "deb http://tonelli.sns.it/pub/mplayer/sarge ./"
> contains the binaries that I usually prepare, for Debian 3.1
> a.k.a. sarge ; these do not contain all the upstream code, and in
> particular do not contain the libmpdvdkit (for fear of patent infringment)
> 
>  "deb http://tonelli.sns.it/pub/mplayer/etch ./"
> the same, for Debian/etch . This I am trying to have into main Debian
> distribution.

I reviewed this one. Here are further comments:

1. Package GUI separately, just like I described in packaging
   guidelines.
2. Don't include uuencoded tar.gz with default skin - find another
   way to do it, maybe separate package?
3. Modify MPlayer's version string as described in packaging guidelines.
4. Don't use an init script to set /proc/sys/dev/rtc/max-user-freq,
   modify /etc/sysctl.conf instead.
5. Package mencoder separately.
6. You shouldn't need to use so many --enable-* options. Just providing
   the correct build dependencies should enable ./configure's
   autodetection to work. The options that need --enable'ing are:
   --enable-largefiles
   --enable-dynamic-plugins
   --enable-joystick
   --enable-menu
   --enable-mga
   --enable-tdfxfb
   --enable-xmga
   --enable-xmms
   --enable-xvmc
   The following may need to be --enable'd to skip device autodetection:
   --enable-directfb
   --enable-fbdev
   --enable-lirc
   --enable-v4l
   --enable-v4l2

Regards,
R.

-- 
MPlayer RPMs maintainer: http://rpm.greysector.net/mplayer/
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"