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

Ivan Kalvachev ikalvachev at gmail.com
Wed Feb 27 00:53:56 CET 2008


On Wed, Feb 27, 2008 at 12:25 AM, Diego Biurrun <diego at biurrun.de> wrote:
> 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.

There are headers in ffmpeg that are installed system-wide, so they
must be able to be included without knowing the "magic" combination.
However MPlayer doesn't install any headers.



More information about the MPlayer-dev-eng mailing list