[MPlayer-dev-eng] [RFC] port 'make checkheaders' from FFmpeg

Diego Biurrun diego at biurrun.de
Tue Feb 26 23:25:41 CET 2008


On Tue, Feb 26, 2008 at 07:49:37PM +0100, Alban Bedel wrote:
> On Tue, 26 Feb 2008 14:25:01 +0100
> Guillaume Poirier <gpoirier at mplayerhq.hu> wrote:
> 
> > Ivan Kalvachev wrote:
> > > On Sun, Feb 24, 2008 at 3:23 PM, Diego Biurrun <diego at biurrun.de>
> > > wrote:
> > >> I've been having this lying around in my local tree for some
> > >> time.  The thread about OS X/Intel warnings reminded me to dig
> > >> this out again since it represents the all-inclusive general
> > >> solution.
> > >>
> > >>  So here are two patches, one to add the checkheaders target to
> > >>  mpcommon.mak and another proof-of-concept one that adds a couple
> > >> of the missing header files to assorted files (no, it's not
> > >> complete).  The idea is to have standalone headers in the end that
> > >> can be used without dependencies.
> > >>
> > >>  I'd like to commit this soon.
> > > 
> > > No
> > 
> > Mmmm.... looks like someone needs to grow up and become an adult....
> 
> Look like someone is still ignoring that some think that having all
> header deps resolved in the source is better. Ivan explaind several
> time why and I agree with him on this point.

Headers should work standalone without the need to include them in some
magic sequence.  This policy is working quite well for the FFmpeg
project.

Diego



More information about the MPlayer-dev-eng mailing list