[MPlayer-dev-eng] Re: [PATCH] improvements in debian/* (Adam Di Carlo)

Dariusz Pietrzak dariush at ajax.umcs.lublin.pl
Thu Dec 20 14:22:46 CET 2001


Hello,

 i've applied most of it, but this:
"
 * debian/control: remove invalid source recommends

-Recommends: libsdl1.2-dev
+Build-Depends: c++-compiler, libglib-dev, libgtk-dev, xlibs-dev,
libpng2-dev, zlib1g-dev, debhelper (>= 2), libmad0-dev, aalib1-dev,
libasound1-dev, libsdl1.2-dev
"
I don't understand.
Neither libsdl nor libmad0 are really needed to build mplayer. If anything
- i would remove dependancy on xlibs-dev etc and don't build gui at all.
OTOH i haven't checked the gui for a long time, i'll fix that.

About postinst - it used to be /bin/sh, but I felt bad with it, since it
was invoking debconf, which was invoking perl.. so decided that since perl
can do postinst, then there's no reason to run /bin/sh.
 Since there is cdebconf, this decision no longer makes much sense even to
me.

-- 
Dariusz Pietrzak
snafu




More information about the MPlayer-dev-eng mailing list