[MPlayer-dev-eng] [PATCH] Mac OS X Application Bundle Support

D Richard Felker III dalias at aerifal.cx
Thu Nov 11 06:38:03 CET 2004


On Thu, Nov 11, 2004 at 04:15:47AM +0100, Diego Biurrun wrote:
> Chris Roccati writes:
> > 
> > the patch, basically changes the macros MPLAYER_CONFDIR, 
> > MPLAYER_DATADIR and MPLAYER_LIBDIR into macros taking a single 
> > parameter. So MPLAYER_CONFDIR "/codecs.conf" becomes 
> > MPLAYER_CONFDIR("/codecs.conf"), so the enable-macosx-bundle, can make 
> > the macros to expand as get_resource_path("/codecs.conf").
> 
> And why not simply set reasonable values for DATADIR, CONFDIR and
> LIBDIR with configure parameters?

i agree. unless i'm badly misunderstanding, this patch is
unacceptable. plus, mplayer already has a function to search for
config files in its paths, right?

rich




More information about the MPlayer-dev-eng mailing list